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

update_copyright_year.py & remove trailing whitespace in copyright blocks #994

Merged
merged 29 commits into from Feb 4, 2015

Conversation

remram44
Copy link
Member

I might have gone overboard with that, maybe only fix whitespace in Python code.

@rexissimus
Copy link
Member

It should be fine. But the script only works on .py files for safety. We should run it for other file and update other files like splash.svg and possibly others.

@remram44
Copy link
Member Author

I do not know whether this should be "2011-2015 NYU-Poly" or "2011-2014 NYU-Poly, 2014-2015 New York University" or even "2011-2015 New York University".

@rexissimus
Copy link
Member

Yes, we should discuss this. Also, the text still mentions "University of Utah".

@remram44
Copy link
Member Author

Copyright headers now look like this:

## Copyright (C) 2014-2015, New York University.
## Copyright (C) 2011-2014, NYU-Poly.
## Copyright (C) 2006-2011, University of Utah.
## All rights reserved.
## Contact: contact@vistrails.org

@rexissimus
Copy link
Member

I updated other file types as well and removed "University of Utah" from the copyright text. Also updated the splash and the mac dmg binary. I think that should be everything. Different file types with different formatting and different types of generators makes updating a pain :(

rexissimus added a commit that referenced this pull request Feb 4, 2015
updates copyright year and copyright holder and removes trailing whitespace in copyright blocks
@rexissimus rexissimus merged commit c351e38 into master Feb 4, 2015
@rexissimus rexissimus deleted the copyright-2015 branch February 4, 2015 16:30
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