These are chat archives for scalikejdbc/ja

25th
Mar 2016
kenji yoshida
@xuwei-k
Mar 25 2016 05:39
互換性壊れないように頑張れるなら頑張るに越したことはないけど、 scalikejdbc/scalikejdbc#423 自体を入れることには基本賛成ですね(あまりにも互換性気にしすぎても、ライブラリの進化妨げてアレだし)
Kazuhiro Sera
@seratch
Mar 25 2016 05:44
互換性の一般論としてはそうなんですが、これに関しては SQL#parameters 自体は互換性維持して内部的に ParameterBinder だったりすることがあるというのが良いのでは?と思います。昨日はどうするのがにいいかなと思ってそこまで書かなかったけど。 https://github.com/scalikejdbc/scalikejdbc/pull/423#discussion_r57323255
これはまあしょうがないかなとは思ってるけど、誰か名案があったりしないかなというくらい。 https://github.com/scalikejdbc/scalikejdbc/pull/423#discussion_r57322717
Manabu Nakamura
@gakuzzzz
Mar 25 2016 05:59

うーん、 warning で気付けるかというのはちょっと懐疑的……。

.map(parameterExtractor)SQL#parameters の内部に入れちゃうというアイデアはありかなーと思います。

Kazuhiro Sera
@seratch
Mar 25 2016 11:42

うーん、 warning で気付けるかというのはちょっと懐疑的……。

いや、もしこの PR を入れないなら代替案として考えられるかなというくらいでそうしたいわけではないです。

.map(parameterExtractor) を SQL#parameters の内部に入れちゃうというアイデアはありかなーと思います。

これはそうしてください。