Kazuhiro Sera

Results 522 comments of Kazuhiro Sera

We haven't implemented it yet. I'm open to pull requests from the community! By the way, I revisited this now and came to wonder if there is an equivalent setting...

like stream(where: SQLSyntax): DatabasePublisher[Entity]? It's ok to have it but we should disable(=comment out) the method by default because scalikejdbc-streams is an optional module.

This issue is no longer relevant. Let me close now.

Ah, I overlooked the exclusion settings still exist. Thanks.

The current implementation (version 3.3.1): https://github.com/scalikejdbc/scalikejdbc/blob/3.3.1/scalikejdbc-core/src/main/scala/scalikejdbc/Binders.scala#L168-L172

Thanks for flagging this. > may not override a non-lazy value I'm a bit sad to see this is no longer allowed.

hmm, changing the behavior in a major release may be more appropriate. Perhaps, we should have the change in v4.0.

We won't fix all of these warnings but I will make a pull request that fixes some before releasing the next RC version.

I won't apply any changes in 4.0x. The reasons are: * `scala.reflect.Selectable.reflectiveSelectable` does not exist in Scala 2.13 * Removing any of the `lazy val` in core / config breaks...