Skip to content

Conversation

Kobzol
Copy link
Member

@Kobzol Kobzol commented Sep 30, 2025

After the change of the website to a static web, we:

  • Should not need the deploy branch protection
  • We should require PRs for the main branch (IMO), as it is now automatically deployed
  • The website team should not need maintain permissions anymore, as Pontoon is no longer used
  • We don't need the Heroku bot anymore

CC @senekor if this looks good.

Copy link

github-actions bot commented Sep 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/www.rust-lang.org':
      Permission Changes:
        Changing team 'website''s permission from maintain to write
        Removing user 'rust-heroku-deploy-access''s admin permission 
      Branch Protections:
          main
            Required Approving Review Count: 0 => 1
            Requires PR: false => true
          deploy
            Deleting branch protection

@jieyouxu jieyouxu added needs-infra-admin-review This change requires one of the `infra-admins` to review. S-waiting-on-review Status: waiting on review from a team/WG/PG lead, an infra-admin, and/or a team-repo-admin. labels Oct 1, 2025
@Kobzol
Copy link
Member Author

Kobzol commented Oct 1, 2025

Rebased.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs-infra-admin-review This change requires one of the `infra-admins` to review. S-waiting-on-review Status: waiting on review from a team/WG/PG lead, an infra-admin, and/or a team-repo-admin.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants