These are chat archives for scalikejdbc/ja

7th
Mar 2016
kenji yoshida
@xuwei-k
Mar 07 2016 10:07
あまり状況よくわかってないけど、code-generatorで ParameterBinderFactory[Any] をimplicitのところで明示的に渡すコードを生成するとか?
Kazuhiro Sera
@seratch
Mar 07 2016 10:10
そんな重要なユースケースじゃないですよね。そんな感じでいいんじゃないかと。
今月中には merge してリリースしたいですね。
Manabu Nakamura
@gakuzzzz
Mar 07 2016 10:20

現状、そんな感じの修正をしてコミットしました。
scalikejdbc/scalikejdbc@f8c3f4e

いま scripted-test で mapper-generator の TwentyTree 関連で落ちてるっぽいのですが、その辺まわりいじったつもりが無いのでどうしたものかな、と。
https://travis-ci.org/scalikejdbc/scalikejdbc/jobs/113835282
パラメータが23個なのに class にならずに case class になっちゃってるっぽい?

この辺、ぱっと心当たりみつかったりしますかね?

kenji yoshida
@xuwei-k
Mar 07 2016 10:29
よく見てないけど、わざとコンパイル失敗するテストしてる箇所はあるので、元からそういうテストで、単に失敗してるように見えて成功してるかも(?) https://github.com/scalikejdbc/scalikejdbc/blob/508fcc207afe8e5a7588cb7dac8596ba44ba1bf6/scalikejdbc-mapper-generator/src/sbt-test/scalikejdbc-mapper-generator/twenty-three/test#L15
-> は失敗を期待するテスト http://eed3si9n.com/ja/testing-sbt-plugins
Manabu Nakamura
@gakuzzzz
Mar 07 2016 12:20
失敗を期待してる感じでは無さそうな気配……。むむむ。
https://travis-ci.org/scalikejdbc/scalikejdbc/jobs/114216324#L7188
Manabu Nakamura
@gakuzzzz
Mar 07 2016 12:32
単に直し方がまずかったっぽい気配
Manabu Nakamura
@gakuzzzz
Mar 07 2016 12:40
引き続き GitHub上で雑に編集してcommit しております。(scripted-test ローカルのWindows上で動かすの諦めました)
Manabu Nakamura
@gakuzzzz
Mar 07 2016 13:49
お、 scripted-test 通った気配
kenji yoshida
@xuwei-k
Mar 07 2016 13:50
あーなるほどwindows無理だったっけ
Manabu Nakamura
@gakuzzzz
Mar 07 2016 13:59
テスト通ったっぽい~
のでコードレビューお願いしますm(_ _)m
  • https://github.com/scalikejdbc/scalikejdbc/pull/423/files#diff-16b0d646d677336eaf1a1f35b90a80deR29
    この辺の toString、 ParameterBinder() とか出力せずに value の値そのままの文字列にした方がログとかの互換性維持できそうだけどどうしよう?
  • ParameterBinderFactory は本当に bimap でいいのか、とか(本質的には contramap だけでいいはずだけどログの書き出しとかのために戻す処理まで入れるべきなのか)
などなどご意見お待ちしております
Manabu Nakamura
@gakuzzzz
Mar 07 2016 14:05
あと各種Scaladocコメントに書く英文とか
Manabu Nakamura
@gakuzzzz
Mar 07 2016 17:09
ParameterBinder#mapprivate[scalikejdbc] にするべきか……