You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Unless I'm missing something, this makes it difficult to use a custom header as it requires users to place the custom header in io/spring/javaformat/checkstyle/check package on their Checkstyle classpath. It would be nice if SpringHeaderCheck would load the custom header from the root of Checkstyle classpath.
To clarify, assuming a Gradle build with default Checkstyle config location of config/checkstyle one need to do the following:
And then place the custom header file in config/checkstyle/io/spring/javaformat/checkstyle/check while the desirable would be to simply place it in config/checkstyle.
The text was updated successfully, but these errors were encountered:
Actually, on review the file isn't really designed to be replaced in that way. It's working by accident currently. I think it would be better if you could specify an external file directly and we ignore the type attribute.
philwebb
changed the title
Consider making SpringHeaderCheck easier to customize
Allow SpringHeaderCheck to load custom files
Mar 30, 2019
Currently,
SpringHeaderCheck
loads header file from the same package:spring-javaformat/spring-javaformat/spring-javaformat-checkstyle/src/main/java/io/spring/javaformat/checkstyle/check/SpringHeaderCheck.java
Lines 146 to 147 in 1830776
Unless I'm missing something, this makes it difficult to use a custom header as it requires users to place the custom header in
io/spring/javaformat/checkstyle/check
package on their Checkstyle classpath. It would be nice ifSpringHeaderCheck
would load the custom header from the root of Checkstyle classpath.To clarify, assuming a Gradle build with default Checkstyle config location of
config/checkstyle
one need to do the following:And then place the custom header file in
config/checkstyle/io/spring/javaformat/checkstyle/check
while the desirable would be to simply place it inconfig/checkstyle
.The text was updated successfully, but these errors were encountered: