New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add Missing License Headers #1215
base: master
Are you sure you want to change the base?
Conversation
Original author(s) of the
|
The original header contained the year added, but not the current year. This commit fixes it.
I think in etc/jgrapht_checks.xml we need to remove the BeforeExecutionExclusionFileFilter so that module-info.java gets checked as well. |
Something that I missed. Thanks for catching 😄 |
This reverts commit bd4bdd9.
Turns out, we cannot enable Checkstyle running on |
This PR adds the missing license headers to the
module-info.java
andpackage-info.java
files and updates the Checkstyle rule accordingly.The added license headers respect (or at least try to respect) the original authorships of the files, i.e., the year that the file was added and the original author's name are included, not my name and 2024.
I added unit testsHISTORY.md
orCONTRIBUTORS.md