Skip to content

Backport rust-lang-nursery repos (batch 3) #1799

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

Merged
merged 1 commit into from
May 30, 2025

Conversation

Kobzol
Copy link
Contributor

@Kobzol Kobzol commented Apr 30, 2025

Backporting the following repos:

  • .github

    Original content
    org = "rust-lang-nursery"
    name = ".github"
    description = "GitHub metadata for this organization"
    bots = []
    
    [access.teams]
    
    [access.individuals]
    erickt = "admin"
    KodrAus = "admin"
    nikomatsakis = "admin"
    aidanhs = "admin"
    alexcrichton = "admin"
    kennytm = "admin"
    huonw = "admin"
    pietroalbini = "admin"
    steveklabnik = "admin"
    carols10cents = "admin"
    sfackler = "admin"
    nrc = "admin"
    marcoieni = "admin"
    Mark-Simulacrum = "admin"
    jdno = "admin"
    rust-lang-owner = "admin"
    nikomatsakis-admin = "admin"
    aturon = "admin"
    dtolnay = "admin"
  • rust-lang-nursery.github.io

    Original content
    org = "rust-lang-nursery"
    name = "rust-lang-nursery.github.io"
    description = "GitHub pages redirects"
    bots = []
    
    [access.teams]
    
    [access.individuals]
    Mark-Simulacrum = "admin"
    carols10cents = "admin"
    steveklabnik = "admin"
    aidanhs = "admin"
    nrc = "admin"
    huonw = "admin"
    pietroalbini = "admin"
    erickt = "admin"
    dtolnay = "admin"
    nikomatsakis = "admin"
    kennytm = "admin"
    aturon = "admin"
    KodrAus = "admin"
    jdno = "admin"
    nikomatsakis-admin = "admin"
    marcoieni = "admin"
    rust-lang-owner = "admin"
    alexcrichton = "admin"
    sfackler = "admin"
  • rust-toolstate

    Original content
    org = "rust-lang-nursery"
    name = "rust-toolstate"
    description = "Records build and test status of external tools bundled with the Rust repository."
    bots = []
    
    [access.teams]
    bots = "write"
    infra = "admin"
    
    [access.individuals]
    Mark-Simulacrum = "admin"
    alexcrichton = "admin"
    marcoieni = "admin"
    kennytm = "admin"
    aturon = "admin"
    nrc = "admin"
    sfackler = "admin"
    pietroalbini = "admin"
    dtolnay = "admin"
    shepmaster = "admin"
    erickt = "admin"
    jdno = "admin"
    huonw = "admin"
    carols10cents = "admin"
    KodrAus = "admin"
    rust-highfive = "write"
    nikomatsakis-admin = "admin"
    aidanhs = "admin"
    Kobzol = "admin"
    steveklabnik = "admin"
    nikomatsakis = "admin"
    pietroalbini-cat = "write"
    rust-lang-owner = "admin"
    bors = "write"
    
    [[branch-protections]]
    pattern = "master"
    required-approvals = 0
    pr-required = false

Not completely sure about rust-toolstate, I added access to highfive, it seems like bors access shouldn't be needed?

@Kobzol Kobzol force-pushed the rust-lang-nursery-3 branch from bb4aa8f to ebd0bfc Compare April 30, 2025 13:08
@Kobzol Kobzol marked this pull request as ready for review April 30, 2025 13:08
@Kobzol Kobzol requested a review from Mark-Simulacrum April 30, 2025 13:08
@Kobzol Kobzol force-pushed the rust-lang-nursery-3 branch 2 times, most recently from d81f934 to a7e38f5 Compare April 30, 2025 13:38
Copy link

github-actions bot commented Apr 30, 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-nursery/.github':
      Enable auto-merge
      Permission Changes:
        Giving team 'infra' write permission
    📝 Editing repo 'rust-lang-nursery/rust-lang-nursery.github.io':
      Remove homepage: ''
      Enable auto-merge
      Permission Changes:
        Giving team 'infra' write permission
    📝 Editing repo 'rust-lang-nursery/rust-toolstate':
      Enable auto-merge
      Permission Changes:
        Changing team 'infra''s permission from admin to write
        Removing team 'bots''s write permission 
        Removing user 'pietroalbini-cat''s write permission 
      Branch Protections:
          master
            Is admin enforced: false => true

@Kobzol Kobzol force-pushed the rust-lang-nursery-3 branch from a7e38f5 to 3432e06 Compare April 30, 2025 18:10
@marcoieni
Copy link
Member

    📝 Editing repo 'rust-lang-nursery/rust-toolstate':
      Branch Protections:
          master
            Is admin enforced: false => true

are you sure that toolstate will keep working if we do this?

@Kobzol
Copy link
Contributor Author

Kobzol commented May 29, 2025

It's hard to say I'm sure of anything with these GitHub settings 😆 But I don't see why it should fail, highfive shouldn't be abusing any admin privileges, AFAIK, bots are only given write access. So I think it should be fine.

@marcoieni marcoieni added this pull request to the merge queue May 30, 2025
Merged via the queue into rust-lang:master with commit 0eb9f82 May 30, 2025
3 checks passed
@Kobzol Kobzol deleted the rust-lang-nursery-3 branch May 30, 2025 08:22
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.

2 participants