>has one blank line after copyright

Beam seems to have mixed style as well:
blank line before "package":
https://github.com/apache/beam/blob/166c6de33f2491c4c9bd27511cc71e33f8d2a894/buildSrc/src/main/groovy/org/apache/beam/gradle/GrpcVendoring.groovy#L18
no blank before "package":
https://github.com/apache/beam/blob/a2b0ad14f1525d1a645cb26f5b8ec45692d9d54e/examples/java/src/main/java/org/apache/beam/examples/subprocess/configuration/SubProcessConfiguration.java#L17-L18

The inconsistency with style across different files is sad.

>I did a test.

Did you use package-info.java for the test?

Here's my test:
$ git diff
diff --git
a/kafka/src/main/java/org/apache/calcite/adapter/kafka/package-info.java
b/kafka/src/main/java/org/apache/calcite/adapter/kafka/package-info.java
index 8805de348..77f66752d 100644
--- a/kafka/src/main/java/org/apache/calcite/adapter/kafka/package-info.java
+++ b/kafka/src/main/java/org/apache/calcite/adapter/kafka/package-info.java
@@ -1,7 +1,6 @@
 /*
  * Licensed to the Apache Software Foundation (ASF) under one or more
- * contributor license agreements.  See the NOTICE file distributed with
- * this work for additional information regarding copyright ownership.
+ asdf lkasjhdf lkasjdhf lkasjhdlk h
  * The ASF licenses this file to you under the Apache License, Version 2.0
  * (the "License"); you may not use this file except in compliance with
  * the License.  You may obtain a copy of the License at

$ gw :kafka:spotlessCheck
Building Apache Calcite 1.22.0-SNAPSHOT
...
BUILD SUCCESSFUL in 1s
2 actionable tasks: 1 executed, 1 up-to-date

Vladimir

Reply via email to