These are chat archives for shirasagi/shirasagi

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

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

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

task :foo => :environment do
end

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

task foo: :environment do
end

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

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