dweiss commented on a change in pull request #1950:
URL: https://github.com/apache/lucene-solr/pull/1950#discussion_r501492773



##########
File path: gradle/validation/spotless.gradle
##########
@@ -0,0 +1,31 @@
+
+def resources = scriptResources(buildscript)
+ 
+allprojects { prj ->
+  plugins.withType(JavaPlugin) {
+    prj.apply plugin: 'com.diffplug.spotless'
+
+    spotless {
+      java {
+        licenseHeaderFile file("${resources}/asl-header.txt"), '^(\\s*package)'
+        lineEndings 'UNIX'
+        endWithNewline()
+        googleJavaFormat('1.9')
+
+        // Known problematic files.
+        targetExclude "**/HTMLStripCharFilter.java", 
"**/UAX29URLEmailTokenizerImpl.java", 
+           "**/PatternParser.java", "**/BuildNavDataFiles.java", 
"**/CheckLinksAndAnchors.java",
+           "**/TestSubQueryTransformer.java"

Review comment:
       Ok, fair enough. I also suggested elsewhere that we might want to just 
move those generated files into a different sourceset so that it's clear they 
are generated and they can be wiped easier upon regeneration. Or is there value 
in keeping them together, do you think?




----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org
For additional commands, e-mail: issues-h...@lucene.apache.org

Reply via email to