Skip to content

Conversation

@marcoieni
Copy link
Member

@marcoieni marcoieni commented Oct 23, 2025

Is this enough? Or some other repo depends on master?

Related to rust-lang/infra-team#227

before merging this:

@github-actions
Copy link

github-actions bot commented Oct 23, 2025

Dry-run check results

[WARN  sync_team] sync-team is running in dry mode, no changes will be applied.
[INFO  sync_team] synchronizing github
[INFO  sync_team] 💻 Repo Diffs:
    📝 Editing repo 'rust-lang/team':
      Branch Protections:
          main
            Dismiss Stale Reviews: true
            Is admin enforced: true
            Required Approving Review Count: 1
            Requires PR: true
            Required Checks: ["CI"]
            Allowances: []
          master
            Deleting branch protection

Copy link
Member

@Kobzol Kobzol left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You should also modify the branch protection in repos/rust-lang/team.toml.

A bunch of repos depend on team through git, but Cargo should be using HEAD instead of master for quite some time, so I think that should be fine.

@jieyouxu jieyouxu added the T-infra Relevant to the infrastructure team. label Oct 24, 2025
@marcoieni marcoieni force-pushed the rename-default-branch-to-main branch from f759538 to b273912 Compare October 24, 2025 09:58
@marcoieni
Copy link
Member Author

marcoieni commented Oct 24, 2025

Ok, this is ready to be reviewed by another infra admin 👍
Once this PR is approved I can go through the process of updating the branch

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

T-infra Relevant to the infrastructure team.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants