Skip to content
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

Updated/added license headers of all files according to new configura… #84

Open
wants to merge 1 commit into
base: devel
Choose a base branch
from

Conversation

RadekKoubsky
Copy link
Member

…tion of license-maven-plugin.

See issue #80

I have changed the configuration of license-maven-plugin:

I removed all license headers and add new license headers according to the new configuration using goal mvn license:update-file-header to make all headers consistent.

Building script contains goal license:check-file-header -Dlicense.licenseName=apache_v2 to check missing headers before build.

@RadekKoubsky RadekKoubsky force-pushed the license-maven-plugin-update branch from 548b971 to c4b2ad7 Compare January 24, 2017 09:19
@RadekKoubsky RadekKoubsky force-pushed the license-maven-plugin-update branch from c4b2ad7 to dc3b4b7 Compare February 4, 2017 23:16
@RadekKoubsky RadekKoubsky force-pushed the license-maven-plugin-update branch from dc3b4b7 to 60ea60b Compare February 4, 2017 23:51
@marvec
Copy link
Member

marvec commented Feb 8, 2017

The headers were intentional because the plugin did not recognize the headers properly without them.

@marvec
Copy link
Member

marvec commented Feb 8, 2017

They were there to fix a problem. So unless you figure out what problem it was and come up with a solution that fixes both issues, we cannot merge this, sorry.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants