-
Notifications
You must be signed in to change notification settings - Fork 328
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
Avoid hardcoding year values #678
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Kinda embarrassed that none of us ever thought of this before
Looks like there are some Rubocop issues blocking the CI. If you could address those, @binford2k, this PR should be an easy review/merge. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Couple of wee adjustments needed before we can merge.
Fixes puppetlabs#660 long term
Co-authored-by: SE-Stephen P Potter <56322084+spotter-puppet@users.noreply.github.com>
Co-authored-by: SE-Stephen P Potter <56322084+spotter-puppet@users.noreply.github.com>
Co-authored-by: Lukas Audzevicius <97180854+LukasAud@users.noreply.github.com>
eb1fda8
to
f6d6003
Compare
Fixes #660 long term