-
Notifications
You must be signed in to change notification settings - Fork 87
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
Eliminate settings.SENIOR_GRADUATION_YEAR #989
Conversation
What about the edge case where this is no graduation this year 😢 |
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.
In addition to fixing the build (after regressions in pylint are resolved), please make the world eliminate
in the commit message lower case consistent with https://github.com/tjcsl/ion/blob/master/docs/developing/howto.rst#formatting-commit-messages.
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.
Please rebase to incorporate recent changes to dev
.
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 once build passes! Thanks for your contribution to Ion!
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.
Forgot to mention that eliminate should be lowercase in the commit message.
In favor of auto-detection based on current date.
Proposed changes
settings.SENIOR_GRADUATION_YEAR
in favor of auto-detection based on current date.Brief description of rationale
Updating
settings.SENIOR_GRADUATION_YEAR
is annoying.