-
Notifications
You must be signed in to change notification settings - Fork 33
Add travis.yml #206
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
Add travis.yml #206
Conversation
nvcheck に v0.1 タグを打って、gopkg 経由で使うよにしたほうが、良いですかね? 出力周りとか、手を入れたいので。 |
というか、打っておきました。 https://github.com/koron/nvcheck/releases/tag/v0.1 |
あれ?逆か? |
CIでは、masterの存在が、邪魔になりそうだな (;´∀`) |
なんか dict のキーと値が逆になってて正しいのが出てる気がする。 |
vim から errorformat で使える様にしたいですね。 |
たしかに、なんかおかしい |
以下提案ですが、nvcheck の報告を無視したい例外的なケースもその内出てくるでしょうから、出力するログを channel.jax サーバ >> サーバー at line 34 のように単一行で完結するようにして、出力されたログをそのまま .nvcignore (仮称)のようなファイルに追記すれば無視される仕組みにするというのはいかがでしょうか? Travis CI が出力した nvcheck のログをファイルに追記して commit するだけなら Go の実行環境が必須ではなくなりますので、作業の敷居が低くなるはずです。 行の挿入等で行番号にズレが生じると、単に .nvcignore (仮称)に追記するだけではゴミが残りますが、その場合は手元で nvcheck を実行できる人が時々掃除するという運用でカバーするしかなさそうですね。 |
いま nvcheck のほうはこんな風に出力するように、修正中です。
|
これ、辞書微妙に間違ってませんかね? どこから作ったのかしら? |
9e9808b でちゃんとした dict.yml を作ってみました。 |
いったん全部結果出して異常終了する方が良さそうすね。後で nvcheck.sh 弄ります。 |
@mattn ファッ?! |
語句は以下から抽出した。 #37 (comment) 内容については要ダブルチェック。 nvcheck は https://github.com/koron/nvcheck を参照。
そろそろマージしても良い感じ? |
🙆♀️ |
カレントから全部探すのは、なにかしら問題になることもありそうなので、探す 場所を限定しました。
thx all! |
マージしないでください。