Skip to content

Commit

Permalink
Don't skip existing license headers if they don't match the format.
Browse files Browse the repository at this point in the history
  • Loading branch information
jkschneider authored Dec 12, 2024
1 parent 7aefd4d commit 72f5a45
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,7 @@ public void apply(Project project) {
});

project.getExtensions().configure(LicenseExtension.class, ext -> {
ext.setSkipExistingHeaders(true);
ext.setSkipExistingHeaders(false);
ext.getExcludePatterns().addAll(Arrays.asList("**/*.tokens", "**/*.config", "**/*.interp", "**/*.txt", "**/*.bat",
"**/*.zip", "**/*.csv", "**/gradlew", "**/*.dontunpack", "**/*.css",
"**/*.editorconfig", "**/*.md", "**/*.jar"));
Expand Down

1 comment on commit 72f5a45

@timtebeek
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As a side effect we seem to now override the copyright year on existing headers; which we then promptly enforce on pull requests:
openrewrite/rewrite#4738 (review)

All that is fine for now, but perhaps best reevaluated in 2025 when this will again prompt to update all license headers.

Please sign in to comment.