chetanmeh edited a comment on issue #3928: Replace List() occurrences with List.empty URL: https://github.com/apache/incubator-openwhisk/pull/3928#issuecomment-409273589 Given that this is a recurring work (see #3359) should we look into configuring a code style check plugin. For example [scapegoat][1] has a `PreferSeqEmpty` check which can flag such usage. Only issue is I [do not find a Gradle plugin](sksamuel/scapegoat#195) for it. But looking at [sbt plugin][2] it should be possible to add it directly via some scripting in our case [1]: https://github.com/sksamuel/scapegoat [2]: https://github.com/sksamuel/sbt-scapegoat/blob/master/src/main/scala/com/sksamuel/scapegoat/sbt/ScapegoatSbtPlugin.scala
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services