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

Allow options and parameters in any order #521

Merged
merged 1 commit into from
Feb 21, 2024

Conversation

ehelms
Copy link
Member

@ehelms ehelms commented Aug 24, 2021

No description provided.

@upadhyeammit
Copy link
Contributor

The latest version of clamp https://rubygems.org/gems/clamp/versions/1.3.2 is not available in non scl maintenance repository. I can see rubygem-clamp-0.6.2-2. is doing duties now. So this means we need to bump clamp and required deps in maintenance repository too.

@ehelms
Copy link
Member Author

ehelms commented Sep 1, 2021

Good catch, I can look at if we could upgrade clamp. Or wrap this in a version check (for EL8 foreman-maintain) or wait to see if we SCL foreman-maintain if we drop older Ruby.

@ehelms ehelms marked this pull request as draft December 7, 2021 13:20
@ehelms ehelms closed this Mar 31, 2023
@ehelms ehelms reopened this Feb 21, 2024
@ehelms ehelms marked this pull request as ready for review February 21, 2024 01:35
@ehelms
Copy link
Member Author

ehelms commented Feb 21, 2024

We now use the right version of Clamp! This allows for things that do not work to work.

Prior to change:

satellite-maintain packages install -y ansible-collection-redhat-satellite

After change:

satellite-maintain packages install ansible-collection-redhat-satellite -y

@ehelms ehelms merged commit b22c70b into theforeman:master Feb 21, 2024
5 of 6 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