-
Notifications
You must be signed in to change notification settings - Fork 79
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
Fix source publish bug #1116
Fix source publish bug #1116
Conversation
Fix a source publishing bug where the code was looping through all release components instead of just the release components for the distribution when attempting to publish source files for a specific distribution component. This raised a key error that was found in @acheng-01's PR which added a test for source publishing: pulp#1061 [noissue]
8f2cc7a
to
ccd05a1
Compare
Thanks for identifying this! |
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.
Uh, oh. I missed that! Sorry for the inconvenience! Thank you, good catch!
Backport to 3.2: 💚 backport PR created✅ Backport PR branch: Backported as #1131 🤖 @patchback |
Backport to 3.3: 💚 backport PR created✅ Backport PR branch: Backported as #1132 🤖 @patchback |
Fix source publish bug (cherry picked from commit 0a05615)
Fix source publish bug (cherry picked from commit 0a05615)
…9f6f15687268a1bccbac82ccc99332/pr-1116 [PR #1116/0a05615e backport][3.2] Fix source publish bug
…9f6f15687268a1bccbac82ccc99332/pr-1116 [PR #1116/0a05615e backport][3.3] Fix source publish bug
Fix a source publishing bug that was introduced in b29542e where the code was looping through all release components instead of just the release components for the distribution when attempting to publish source files for a specific distribution component. This raised a key error that was found in @acheng-01's PR #1061 which added a test for source publishing.