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

feat: Eliminate command 'remove' from subscription-manager #3444

Merged
merged 1 commit into from
Sep 4, 2024

Conversation

jvlcek
Copy link
Contributor

@jvlcek jvlcek commented Aug 6, 2024

This PR eliminates the subscription-manager remove sub-command.

fixes: https://issues.redhat.com/browse/CCT-426

@jvlcek jvlcek changed the title [WIP] feat: Eliminate command 'remove' from subscription-manager feat: Eliminate command 'remove' from subscription-manager Aug 8, 2024
@jvlcek jvlcek force-pushed the CCT-426_remove_remove branch from 75b23f3 to 069cc19 Compare August 8, 2024 15:43
Copy link
Contributor

@jirihnidek jirihnidek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ptoscano
Copy link
Contributor

Please rebase and fix the conflicts due to the other removals, thanks!

@ptoscano
Copy link
Contributor

Please rebase and fix the conflicts due to the other removals, thanks!

... and in addition to that, please squash the commit fixing the changes to the man page together with the original commit

@jvlcek jvlcek force-pushed the CCT-426_remove_remove branch 2 times, most recently from 8b6625d to f8ce2a9 Compare September 3, 2024 19:17
This PR eliminates the subscription-manager remove sub-command.

fixes: https://issues.redhat.com/browse/CCT-426
@jvlcek jvlcek force-pushed the CCT-426_remove_remove branch from 77353be to 027dd3d Compare September 3, 2024 19:42
@jvlcek
Copy link
Contributor Author

jvlcek commented Sep 3, 2024

I believe this is ready to merge now. @jirihnidek and @ptoscano please take a look.

Thank you! Joe

@ptoscano ptoscano merged commit d9204e3 into candlepin:main Sep 4, 2024
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants