These are chat archives for shirasagi/shirasagi

22nd
Jan 2015
NAKANO Hideo
@sunny4381
Jan 22 2015 04:02
master に push しても codeclimete に反映されないんですが、こんなもんですか?
ka
@kaosf
Jan 22 2015 04:04
少し時間が掛かると思います
今更新されたようです
ka
@kaosf
Jan 22 2015 04:22
rspec で expet(foo).not_to じゃなくて expect(foo).to_not って書き方もあるって初めて知りました.
NAKANO Hideo
@sunny4381
Jan 22 2015 04:23
だいぶ頑張ったのに、判定は C。
:worried:
ka
@kaosf
Jan 22 2015 04:26
complex と言われているのは,ひとまずしょうがないと思います.あれ以上簡単に出来るかな… リファクタリングのためにテストが追加されていたのが :+1:
NAKANO Hideo
@sunny4381
Jan 22 2015 04:37
rubyは緩いですからね。
神山.rbで、JavaとRubyをいったり来たりしている開発者の方が、Rubyはテストがないと大変な事になるといってたのが印象的でした。
NAKANO Hideo
@sunny4381
Jan 22 2015 05:19

Travis CI で

Coveralls encountered an exception:
RestClient::UnprocessableEntity
422 Unprocessable Entity
...
322{"message":"Couldn't find a repository matching this job.","error":true}

というエラーが発生して、coverall に push できていないようです。

次のチケットと関連があるかも?
teikei/teikei#55
ka
@kaosf
Jan 22 2015 06:50
@tany Ezine::SentLogpage_id があるのは分かるとして node_id って何のために必要なんでしたっけ?
tany
@tany
Jan 22 2015 13:17
@sunny4381 Climateの画面で右上の更新マークをクリックすると即時チェックしてくれます
@kaosf 必須ではないですね
ka
@kaosf
Jan 22 2015 13:19
@tany 了解です.
NAKANO Hideo
@sunny4381
Jan 22 2015 13:19
@tany 今度やってみます。
tany
@tany
Jan 22 2015 13:24
@itowtips Arrayフィールドは改行区切りの方がいいかなと。
あとはインポート前に更新がかかるデータのプレビューができた方がよいのと、インポートするときのプライマリキーの指定が必要。
ka
@kaosf
Jan 22 2015 13:31

lib/tasks/**/*.rb 内で

task :foo => :environment do
end

という書き方があるんですけど新しく作る物は

task foo: :environment do
end

で行った方がいいですか?既存のものに合わせておいた方がいいですか?

tany
@tany
Jan 22 2015 13:35
@kaosf 新しい方の書き方がいいと思いますが、あまり見かけないですね..
ka
@kaosf
Jan 22 2015 13:36
絶妙なところだと思います… とりあえず既存のものに合わせておくってことにしておきます.
tany
@tany
Jan 22 2015 13:36
了解です
@itowtips それとグループやロールが存在しないときの処理について
tany
@tany
Jan 22 2015 14:46
@kaosf node_id メルマガコンテンツ内での送信履歴を一括で見るならあった方が便利ですかね