-
Notifications
You must be signed in to change notification settings - Fork 42
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
Update RPM commands for new 3.13 auto-publish #252
Conversation
fca3e71
to
e1f15d3
Compare
fixes: #8759 Required PR: pulp/pulp-cli#252
@@ -60,6 +60,7 @@ def distribution(ctx: click.Context, pulp_ctx: PulpContext, distribution_type: s | |||
update_options = [ | |||
click.option("--base-path"), | |||
click.option("--publication"), | |||
click.option("--repository", callback=_repository_callback), |
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.
This is def more than just adjusted tests.
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.
Well the option was missing, so I added 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.
And it's needed for the autopublish operations?
Then we can reflect this as a feature in the changelog. Like "Adjusted autopublish to new version of ...".
CHANGES/251.bugfix
Outdated
@@ -0,0 +1 @@ | |||
Updated RPM auto-publish tests for future versions >=3.13.0.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.
The changelog is meant for end users so test changes don't need to be called out. I would probably just repurpose this for the repository option as @mdellweg suggests.
e1f15d3
to
c1e385f
Compare
fixes: #8759 Required PR: pulp/pulp-cli#252
fixes: #251