These are chat archives for scalikejdbc/ja

30th
Mar 2016
kenji yoshida
@xuwei-k
Mar 30 2016 07:13
scalikejdbc/scalikejdbc#509 1だけ気が向いたらやる、たぶん
2は project/scalikejdbc.properties からは設定できないし、コマンドからも現状出来ないけど、build.sbtで設定すれば可能なので、ひとまずそれで我慢してくれ、って誰か英語で説明を・・・(?)
2、もっと簡単にsuffixかprefix設定可能にするにしても、内部の整合性考えると面倒な気がする(少なくともissue起票者の人が言ってる方法微妙なのでやりたくない)
kenji yoshida
@xuwei-k
Mar 30 2016 07:36
3 ( テスト生成したくない場合) はもしドキュメント書いてなかったら(or わかりずらかったら)書いておくべきか?
Kazuhiro Sera
@seratch
Mar 30 2016 07:40

2は project/scalikejdbc.properties からは設定できないし、コマンドからも現状出来ないけど、build.sbtで設定すれば可能なので、ひとまずそれで我慢してくれ、って誰か英語で説明を・・・(?)

これはまあ実装した後で私が「こんな感じでいきましょう」とサマればいいと思うので、実装だけしてもらえるならそれはそれでありがたいです。わざわざ議論にしたくない。

3 ( テスト生成したくない場合) はもしドキュメント書いてなかったら(or わかりずらかったら)書いておくべきか?

generator.testTemplate=some_invalid_test_template_name は work around ですからねぇ。ドキュメントすべきか微妙なような。

kenji yoshida
@xuwei-k
Mar 30 2016 07:58
2の実装はなぁ、個人的には (propertiesやコマンドから不可能とはいえbuild.sbtから可能なので) それほどしたくない。
suffix、prefix以外で似たような需要(かつそれらと直交しない設定)が出てきたらキリがないし、だから汎用的に String => String な関数を設定可能にしたので
kenji yoshida
@xuwei-k
Mar 30 2016 08:16
1、たぶん出来た(もう少し確認したらpull reqする)けど、これBooleanでいいのかな(今後の変更に備えてもっと汎用的なかたちにしておけるか?みたいな意味で) xuwei-k/scalikejdbc@9ee9565
kenji yoshida
@xuwei-k
Mar 30 2016 08:45
出した scalikejdbc/scalikejdbc#510
kenji yoshida
@xuwei-k
Mar 30 2016 09:40
面倒だったので 「viewの場合更新系メソッド生成しない」 などはやってない