mdedetrich commented on code in PR #12475: URL: https://github.com/apache/kafka/pull/12475#discussion_r957547580
########## checkstyle/.scalafmt.conf: ########## @@ -12,7 +12,10 @@ # WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. # See the License for the specific language governing permissions and # limitations under the License. -docstrings = JavaDoc +version = 3.5.9 Review Comment: So I checked and its actually a requirement of the newer versions of Scalafmt, if you don't have the version there it complains with ``` Caused by: java.util.NoSuchElementException: version [expected 3.5.9 or 3.5.9]: missing at metaconfig.Configured.get(Configured.scala:15) at com.diffplug.spotless.glue.scalafmt.ScalafmtFormatterFunc.<init>(ScalafmtFormatterFunc.java:46) at java.base/jdk.internal.reflect.DirectConstructorHandleAccessor.newInstance(DirectConstructorHandleAccessor.java:67) ... 130 more ``` Not sure what the reasoning behind having to include the include the scalafmt version in the configuration file but aside from some hacky workaround it needs to be done this away. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: jira-unsubscr...@kafka.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org