-
Notifications
You must be signed in to change notification settings - Fork 46
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
Adjust FileContentSerializer to upstream change #273
Adjust FileContentSerializer to upstream change #273
Conversation
fixes pulp#5428 https://pulp.plan.io/issues/5428 Required PR: pulp/pulp_file#273 Required PR: pulp/pulp-smash#1219
12bee39
to
9fa458c
Compare
fixes pulp#5428 https://pulp.plan.io/issues/5428 Required PR: pulp/pulp_file#273 Required PR: pulp/pulp-smash#1219
fixes pulp#5428 https://pulp.plan.io/issues/5428 Required PR: pulp/pulp_file#273 Required PR: pulp/pulp-smash#1219
fixes pulp#5428 https://pulp.plan.io/issues/5428 Required PR: pulp/pulp_file#273 Required PR: pulp/pulp-smash#1219
fixes #5428 https://pulp.plan.io/issues/5428 Required PR: pulp/pulp_file#273 Required PR: pulp/pulp-smash#1219
CHANGES/5428.misc
Outdated
@@ -0,0 +1 @@ | |||
Adjust FileContentSerializer to upstream change. |
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 probably should be 5428.removal
.
9fa458c
to
4f3ac9f
Compare
I think this is failing because the PR branches it requires got deleted. Will have to remove the Required PRs from the commit I think. |
4f3ac9f
to
e2a6be7
Compare
Let's try. |
@mdellweg I can work with you on merging this. Indeed, let's see if this works or not. |
These travis failures say |
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 looks great thank you @mdellweg !
re #5428
https://pulp.plan.io/issues/5428