Skip to content

Conversation

carols10cents
Copy link
Member

@carols10cents carols10cents commented Sep 3, 2025

Connects to #1687.

This will need someone with the right permissions to actually do the branch rename :)

@senekor
Copy link
Contributor

senekor commented Sep 3, 2025

Maybe @rust-lang/infra has the permissions? (or knows who does...)

@Kobzol
Copy link
Member

Kobzol commented Sep 3, 2025

Yes, this pretty much has to be done by an infra admin. A PR also has to be sent to the team repo to update branch protections.

@marcoieni
Copy link
Member

marcoieni commented Sep 3, 2025

This PR looks good to me.
I'm an infra admin, I can do this change. Here's the plan I propose:

Does the sequence looks right?

@senekor
Copy link
Contributor

senekor commented Sep 3, 2025

Maybe it's possible to protect both master and main in the pattern? Then the team repo PR can be merged first, so there is no "downtime" for the branch protection.

@Kobzol
Copy link
Member

Kobzol commented Sep 3, 2025

I don't think that's needed, but it would also work.

@marcoieni
Copy link
Member

I also don't think it's needed to protect both main and master since I will perform the steps quickly 👍
I will execute the plan once rust-lang/team#1976 is approved. I'm pinging someone

@Kobzol
Copy link
Member

Kobzol commented Sep 4, 2025

Note that promote-release might require an update for this (https://github.com/rust-lang/promote-release/blob/ee6866331f1d61fc9faa5df00bc0305eb7a2ad65/src/main.rs#L37).

@marcoieni
Copy link
Member

marcoieni commented Sep 4, 2025

oh, ok, than: we should wait for rust-lang/promote-release#95 to be deployed first 👍

@Kobzol
Copy link
Member

Kobzol commented Sep 5, 2025

(My PR doesn't fix this, it only removes master references for r-l/r, not the blog.)

@marcoieni
Copy link
Member

oh, I see, so we need to open another PR to fix it 👍

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.

4 participants