Skip to content

Conversation

jieyouxu
Copy link
Member

@jieyouxu jieyouxu commented May 24, 2025

  • 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

Copy link

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

@pietroalbini
Copy link
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
3 checks passed
@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