On Wed, Jun 26, 2019 at 4:22 PM Anton Kedin <ke...@google.com> wrote:
> Currently our spotless is configured globally [1] (for java at least) to > include all source files by '**/*.java'. And then we exclude things > explicitly. Don't know why, but these exclusions are ignored for me > sometimes, for example `./gradlew :sdks:java:core:spotlessJavaCheck` always > fails when checking the generated files under > `.../build/generated-src/antlr/main/org/apache/beam/sdk/schemas/parser/generated`. > > Few questions: > * can someone point me to a discussion or a jira about this behavior? > BEAM-6399 and BEAM-7366 allude to something wonky going on. > * do we actually have a use case of checking the source files that are > not under 'src'? > No > * if not, can we switch the config to only check for sources under 'src' > [2]? > Yes > * alternatively, would it make sense to introduce project-specific > overrides? > All src should be under src/ so it is unlikely to be useful. > > [1] > https://github.com/apache/beam/blob/af9362168606df9ec11319fe706b72466413798c/buildSrc/src/main/groovy/org/apache/beam/gradle/BeamModulePlugin.groovy#L819 > [2] https://github.com/apache/beam/pull/8954 >