search

crystal-jp / introducing-crystal

『Introducing Crystal Programming Language』の原稿をまとめたリポジトリです
https://crystal-jp.github.io/introducing-crystal/
17 stars 3 forks source link

shardsでinstallしたライブラリも `crystal tool format` の対象にされてしまい落ちる #12

Closed at-grandpa closed 6 years ago

at-grandpa commented 6 years ago

CLI ビルダーの https://github.com/mrrooijen/commander をinstallしているのですが、crystal tool format をすると、 

Error: '/path/to/techbookfest4/08-cli-development/projects/myecho_commander/lib/commander/spec/commander_spec.cr' has syntax errors

と落ちてしまいます。libディレクトリ配下は format の対象にしないほうが良さそうですね。

ちょっと対策考えます。パッチ思いつく方は大歓迎です！

at-grandpa commented 6 years ago

make test の前に make lint が走る場合は大丈夫なのですが。。

at-grandpa commented 6 years ago

https://github.com/mrrooijen/commander に PR送りますかね。。

makenowjust commented 6 years ago

このパッチでlib以下は入らなくなる気がする。

diff --git a/Makefile b/Makefile
index be2f594..f4e1d48 100644
--- a/Makefile
+++ b/Makefile
@@ -34,7 +34,7 @@ ADOCS = \
   $(wildcard */*.adoc)
 EXAMPLES = $(wildcard */examples/**/*.cr)
 PROJECTS = $(wildcard */projects/*)
-PROJECT_CRS = $(and $(PROJECTS), $(shell find $(PROJECTS) -name '*.cr'))
+PROJECT_CRS = $(and $(PROJECTS), $(shell find $(PROJECTS) -type d -name lib -prune -o -name '*.cr' -print))
 ASSETS = $(shell find [0-9][0-9]-* -not -name '*.adoc')
 CRS = $(EXAMPLES) $(PROJECT_CRS)
at-grandpa commented 6 years ago

お！行きました！

PR作っておきますねー

makenowjust commented 6 years ago

ボクがやります

at-grandpa commented 6 years ago

お、ではお願いします