Skip to content

Commit

Permalink
Add tips regarding teams to prevent last confusion (#464)
Browse files Browse the repository at this point in the history
Last time, when I tried to update the `model_transparency` repo, I added
a `model_transparency` team and added it to the repo in the same PR
(#454), with a plan to fix team ID in the next one (#456). This actually
caused a problem: the `model_transparency_codeowners` team lost access
to the repository.

This was because the team ID was reused, so Pulumi removed permissions
when creating the new team.

Signed-off-by: Mihai Maruseac <mihaimaruseac@google.com>
  • Loading branch information
mihaimaruseac authored Jun 24, 2024
1 parent 09068f8 commit 07bee5f
Showing 1 changed file with 13 additions and 0 deletions.
13 changes: 13 additions & 0 deletions github-sync/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -81,6 +81,19 @@ example:
permission: push # Must be one of `pull`, `triage`, `push`, `maintain`, or `admin`. Defaults to `pull`.
```
> [!WARNING]
> When adding multiple teams to the same repository, don't duplicate the team
> ID! Doing so might result in loss of permissions to the team with the
> duplicated ID. Instead, skip the ID to the new team if not already known
> (prefer creating a team in one PR and adding it to the repo in a subsequent
> PR).
> [!TIP]
> To get the team ID, inspect the URL of the avatar on the team page (e.g., for
> `https://github.com/orgs/sigstore/teams/model-transparency`, the ID is
> 10329477 as the avatar URL is
> https://avatars.githubusercontent.com/t/10329477).

If you need to configure the GitHub Pages you can add the following definition

```yaml
Expand Down

0 comments on commit 07bee5f

Please sign in to comment.