Skip to content

SUCCESS, FAILを色付きで表示する #66

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Nov 9, 2018

Conversation

lotz84
Copy link
Contributor

@lotz84 lotz84 commented Nov 9, 2018

shell> mmlh verify

のときにSUCCESS, FAILに色がついてたら見やすいかなと思って実装してみました。

ご意見・レビューいただけると 🙏

Before

image

After

image

@igrep
Copy link
Member

igrep commented Nov 9, 2018

ありがたい!
stack ls dependencies | wc -l の結果が #47 の時より大きくなってなければ、今すぐにでもマージしたいぐらいです。。。! 🙏

@lotz84
Copy link
Contributor Author

lotz84 commented Nov 9, 2018

おっしゃる通り、依存関係が増えてしまったことが難点ですね 😓
わかりやすさとのトレードオフだと思うのでHaskellDayが終わったあとに検討いただけると嬉しいです 🙏

@lotz84
Copy link
Contributor Author

lotz84 commented Nov 9, 2018

(CI落ちてしまったので修正します 💦 )

@igrep
Copy link
Member

igrep commented Nov 9, 2018

ああー、そう言えば、ビルドエラーで思い出しましたが、WindowsのコマンドプロンプトだとANSIエスケープコードがうまく表示できないので、その点は気を遣う必要がありますね。。。
cf. #22

@igrep
Copy link
Member

igrep commented Nov 9, 2018

そうそう、CIの出力を見る限り、もともとansi-terminalは孫依存していたみたいなので stack ls dependencies | wc -l の結果は変わらないと思います!よかった!

@igrep igrep merged commit cdb078b into haskell-jp:master Nov 9, 2018
@igrep
Copy link
Member

igrep commented Nov 9, 2018

しまった。Windowsでの問題があるのに思わずマージしてしまった… すみませんが時間があったらrevertします

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants