Skip to content

Grant rust-analyzer team write access to rust-lang/rust - #1841

Merged
Mark-Simulacrum merged 1 commit into
rust-lang:masterfrom
jieyouxu:ra-write-access
May 24, 2025
Merged

Grant rust-analyzer team write access to rust-lang/rust#1841
Mark-Simulacrum merged 1 commit into
rust-lang:masterfrom
jieyouxu:ra-write-access

Conversation

@jieyouxu

@jieyouxu jieyouxu commented May 24, 2025

Copy link
Copy Markdown
Member
  • rust-analyzer already has bors r+ permissions on rust-lang/rust
  • But the team doesn't have write access to rust-lang/rust, which seems very weird.

We should give rust-analyzer team members write access since they already have r+. Noticed while trying to figure out why rust-lang/triagebot#2010 is a problem.

cc @Veykril

@github-actions

Copy link
Copy Markdown
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/rust':
      Permission Changes:
        Giving team 'rust-analyzer' write permission

@emilyalbini

Copy link
Copy Markdown
Member

infra-admins approval given, let's wait for a team repo admin.

@Mark-Simulacrum
Mark-Simulacrum added this pull request to the merge queue May 24, 2025
Merged via the queue into rust-lang:master with commit cfdd9d6 May 24, 2025
@jieyouxu
jieyouxu deleted the ra-write-access branch May 24, 2025 15:05
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.

3 participants