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

Various Cleanup #49

Merged
merged 3 commits into from
Jun 29, 2017
Merged

Various Cleanup #49

merged 3 commits into from
Jun 29, 2017

Conversation

smortex
Copy link
Contributor

@smortex smortex commented Jan 20, 2016

Hello !

In my opinion, the copyright comments on top of each file provides nothing but clutter. What about dropping them (the COPYRIGHT file already says what the user can do, and the VCS information is more accurate than the dates in these comments) ?

I did not touched files in the contrib directory for obvious reasons. Before merging this, I would like to be sure that no contributor gets offended by this. Please follow-up quickly if it's a concern for you !

@smortex smortex added this to the 1.0.0 milestone Jan 20, 2016
@smortex smortex self-assigned this Jan 20, 2016
@neomilium
Copy link
Member

Just for notice, AFAICR, headers were put here accordingly to Debian policy but I'm not able to find any relevant information about that requirement.
I agree this change 👍

The COPYRIGHT file is already here and is more complete, while the VCS
hold accurate information about contributors and conibution date.
@smortex smortex merged commit 626f409 into master Jun 29, 2017
@smortex smortex deleted the cleanup branch June 29, 2017 07:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants