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

License Updates -- *** HUGE CHANGE *** #344

Merged
merged 4 commits into from
Feb 5, 2015
Merged

Conversation

hazendaz
Copy link
Member

@hazendaz hazendaz commented Feb 3, 2015

**** PLEASE READ CAREFULLY. ALL JAVA FILES HAVE BEEN TOUCHED ****

Added apache license badge to project readme.
Move m2e execution filter to parent for cleaner approach.
Add license.txt based on current mybatis java file headers.
Updated license year in POM and the new license.txt
Corrected spelling mistake on ICLA.

The bulk of the change was to update the license headers on all pages. This additionally consisted of a fix with one misplaced license header. This touched all java files. The style of header is now javadoc rather than javascript. Generally only the year changed to include 2015. There were some changes in authors but minimal at best. There was also some minimal spacing changes. This took least approach to get this done. There is a chance I did not see any other duplicated headers like the one I fixed but certainly possible given this level of change.

The license is now driven by presence of the license.txt file in root of project. This is further setup in the parent on a separate pull request. Provided this is liked as a clean solution, I will address getting this into all other mybatis projects. This will activate on all builds so if that is unwanted, it is as simple as deleting the license.txt file now that this has been completed. Then in future years, the file can simply be put back in place. Generally this doesn't appear to slow the builds down very much.

Enjoy!

m2e lifecycle-mapping is needed via plugin defined in parent so move it there instead.

Add license txt file for maven loading licenses
preferrably should be preferably
emacarron added a commit that referenced this pull request Feb 5, 2015
License Updates -- *** HUGE CHANGE ***
@emacarron emacarron merged commit cd320bc into mybatis:master Feb 5, 2015
@emacarron
Copy link
Member

Let's see how this works! Thank you!

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.

3 participants