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

Drop /pulp2 and /streamer from no_proxy_uris #1080

Merged
merged 1 commit into from Aug 26, 2022
Merged

Drop /pulp2 and /streamer from no_proxy_uris #1080

merged 1 commit into from Aug 26, 2022

Conversation

evgeni
Copy link
Member

@evgeni evgeni commented Aug 26, 2022

Includes #1078, because I am lazy and don't want to rebase again.

@evgeni
Copy link
Member Author

evgeni commented Aug 26, 2022

Should this be labeled backwards-incompatible? I think it technically is?

@ekohl
Copy link
Member

ekohl commented Aug 26, 2022

Should this be labeled backwards-incompatible? I think it technically is?

Yes. We're doing a major release anyway so it's just a clearer signal to users reading the changelog.

because I am lazy and don't want to rebase again.

Looks like GH doesn't recognize the commit is redundant. Needs a (trivial) rebase now :(

These were only needed with Pulp2 which we don't support anymore.
@ekohl ekohl enabled auto-merge (rebase) August 26, 2022 15:26
@evgeni
Copy link
Member Author

evgeni commented Aug 26, 2022

Rebased!

Yes, GH doesn't recognize such trivial rebases, but better than getting a vim window again ;)

@ekohl ekohl merged commit 29f2a54 into master Aug 26, 2022
@ekohl ekohl deleted the drop-old-pulp branch August 26, 2022 16:01
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.

None yet

3 participants