apex / apex/up

Crystal support improvements

オープン
#226 コメント 8 件 リアクション 1 件 担当者 0 名 GitHub で見る
主要言語
Go
スター
8.8k
フォーク
383
PR マージ指標
30日以内にマージされた PR はありません

説明

I'd like to improve Crystal support for up - it currently only works with a ``main.cr`` which is older-school. The current standard practice is to have a ``shard.yml`` which describes the app and targets. Suggestions:

1. I fix #172 - yes there is an official image + can move to ``--release``
2. Add yaml parsing, pull out the (first - to start) target from the ``shard.yml`` file and build that instead of forcing ``main.cr``.

FYI: I made a simple demo app with Kemal that imho should "just work" but doesn't due to issue 2 above -- see https://github.com/ukd1/up-crystal-demo - result after running ``up`` on it is this (as it doesn't detect crystal):

screen shot 2017-08-12 at 9 57 00 pm

@tj thoughts?

コントリビューションガイド

コントリビューションガイドを開く

評価

この issue はまだ評価されていません。

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。