These are chat archives for scalikejdbc/ja

22nd
Jul 2015
kenji yoshida
@xuwei-k
Jul 22 2015 09:41
scalikejdbc-warn.png
scalikejdbcのgeneratorが生成するコード、IDEAで上記のようなこと言われるの微妙に気になるので
  • core側で () を外す(listやsingleは副作用ないはず?)
  • generatorで () をつける
  • そんな細かい変更を今更したくないのでこのままでいい
のあたりの意見を募集
Tsukasa Kitachi
@kxbmap
Jul 22 2015 12:14

core側で () を外す

これ支持なんだけど、そうすると.list()でクエリが実行されるようになるのも微妙だなーと思う。
apply -> fetch とかにrenameするのとセットでどうだろうか

kenji yoshida
@xuwei-k
Jul 22 2015 12:21
あーたしかに・・・
applyっていう名前あれよくないよな
kenji yoshida
@xuwei-k
Jul 22 2015 14:57
そういえば前もそれをひとりで悟って、applyを変えないとどうにもならないし、わざわざ互換性を崩すほどでもないか、と諦めた記憶があったような気がしてきた