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

Let's update year in the header #523

Closed
exper0 opened this issue Jan 18, 2016 · 10 comments
Closed

Let's update year in the header #523

exper0 opened this issue Jan 18, 2016 · 10 comments

Comments

@exper0
Copy link
Contributor

exper0 commented Jan 18, 2016

Right now headers contains

* Copyright (c) 2015 Yegor Bugayenko

It's 2016 already, worth updating

@exper0
Copy link
Contributor Author

exper0 commented Jan 20, 2016

@yegor256 what about this issue? There is even PR #528 already.

@davvd
Copy link

davvd commented Jan 23, 2016

@yegor256 dispatch this issue please, see par.21

@yegor256
Copy link
Owner

@exper0 I'll think about it

@mkordas
Copy link

mkordas commented Feb 9, 2016

@yegor256 we're waiting for your resolution here, as my code review is still open

@mkordas
Copy link

mkordas commented Feb 16, 2016

@yegor256 ping

@yegor256
Copy link
Owner

@mkordas I'm waiting for a similar solution in qulice, so that we can just copy it here. I don't think this ticket is worth a full cycle. it's a minor change and a small check into deployment script. do you remember what is the ticket in qulice with the same problem?

@mkordas
Copy link

mkordas commented Feb 16, 2016

@yegor256 in qulice we had yegor256/qulice#583, but this is custom bash script. Maybe we should create our own HeaderCheck that would replace in license template e.g $YEAR$ with current year? Thanks to that we would have generic solution for all projects using Qulice, no separate tickets needed to bump up years any more.

@mkordas
Copy link

mkordas commented Feb 21, 2016

@yegor256 ping

@yegor256
Copy link
Owner

@mkordas we basically need a small check that will fail Maven build when LICENSE.txt doesn't have 2013-2016. that's it. we don't need to check all files. just one file for one text occurrence.

@yegor256
Copy link
Owner

@mkordas I'm closing this, in favor of #627

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

Successfully merging a pull request may close this issue.

4 participants