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

Drop EoL EL7 support #1112

Merged
merged 1 commit into from
Jan 7, 2025
Merged

Drop EoL EL7 support #1112

merged 1 commit into from
Jan 7, 2025

Conversation

bastelfreak
Copy link
Member

Pull Request (PR) description

This Pull Request (PR) fixes the following issues

REFERENCE.md Outdated
Copy link
Member

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"?

Copy link
Member Author

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.

Copy link
Member

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)

Copy link
Member

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

Copy link
Member

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?

Copy link
Member

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 😝 )

@bastelfreak bastelfreak force-pushed the ol7 branch 2 times, most recently from 550e1a2 to 1fbf582 Compare January 6, 2025 19:54
@evgeni evgeni merged commit 5e1b9b2 into voxpupuli:master Jan 7, 2025
22 checks passed
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.

4 participants