Skip to content
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

Spec exit non zero #198

Closed
wants to merge 7 commits into from
Closed

Spec exit non zero #198

wants to merge 7 commits into from

Conversation

tyru
Copy link
Member

@tyru tyru commented Aug 30, 2014

via #196

#197 で失敗したので再作成しました。

Vimがsegmentation faultしてるのにbuildがpassしてるので、
specのスクリプトでexit codeをチェックするようにしました。

@tyru
Copy link
Member Author

tyru commented Aug 30, 2014

あれ…成功してるな…

@tyru
Copy link
Member Author

tyru commented Aug 30, 2014

あーspec.shはテストがbackgroundで動くからフラグ立てても無意味なのか…

@thinca
Copy link
Member

thinca commented Aug 30, 2014

これ今ステータスどんな感じです?

@tyru
Copy link
Member Author

tyru commented Aug 30, 2014

途中で止まってる感じです…
buildがなぜかpassしちゃっててあれーってなって、頭がちょっとぐらぐらしてきたので中断してました。
せめてコメント書くべきでしたね…すみません。

@tyru
Copy link
Member Author

tyru commented Aug 30, 2014

直接このブランチにツッコミとか修正しちゃって構わないので、
誰か原因分かったらアドバイスほしい…

@thinca
Copy link
Member

thinca commented Aug 30, 2014

いえいえ。なんかもう spec.sh にあまり労力割くべきでもないのかなぁとも思ってます。

@tyru
Copy link
Member Author

tyru commented Aug 30, 2014

というか、同等の機能をspec.batで実現するのが面倒だなという感が…
自分以外同等の機能を実装するつもりでもないのかもしれませんが…
面倒な処理は高水準言語であるVim scriptで書くべきですね!!1

@tyru tyru closed this Sep 7, 2014
@tyru tyru deleted the spec-exit-non-zero branch September 7, 2014 03:58
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.

None yet

2 participants