-
Notifications
You must be signed in to change notification settings - Fork 567
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
Drop EoL EL7 support #1112
Drop EoL EL7 support #1112
Conversation
REFERENCE.md
Outdated
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.
why is this file here? shouldn't it be just generated on module release and "gut is"?
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.
during the release process, we've logic in place to verify that the reference.md is up to date, but it needs to exist.
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.
🤔 ok
but still, this is "Drop EL7", it doesn't touch anything release-worthy.
Should that be part of #1111?
(yes, nitpicking, sorry)
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 Foreman we generate it as part of the release because IMHO we shouldn't include generated files in git. In Vox people did want that and I didn't have the energy to push back on it though I still hate it
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.
Can you at least make it a separate commit?
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.
I moved that comit over to #1111 (people really should not give me push permissions everywhere 😝 )
550e1a2
to
1fbf582
Compare
Pull Request (PR) description
This Pull Request (PR) fixes the following issues