Skip to content
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

A bot account is required to automatically generate changelogs #9244

Open
Spookerton opened this issue Sep 12, 2023 · 0 comments
Open

A bot account is required to automatically generate changelogs #9244

Spookerton opened this issue Sep 12, 2023 · 0 comments
Assignees
Labels
Priority: Low [PR/Issue] This does not need short term resolution. Quality of Life [PR] This improves user experience somehow. Repo [PR/Issue] This relates to tooling, dev configuration, or dependencies.

Comments

@Spookerton
Copy link
Member

Spookerton commented Sep 12, 2023

#9220 failed because explicit write permission sadly does not trump branch protection rules.

Accordingly, to fix this, a bot account should be created and added to the organization as a user that can bypass the PR-required branch protection rule for master on PolarisSS13/Polaris. A PAT with the repo and workflow scopes should be created for it via https://github.com/settings/tokens/new (while logged in as the bot account) and added to the repo's secrets via https://github.com/PolarisSS13/Polaris/settings/secrets/actions/new, and that PAT should then be referenced as the token for the stefanzweifel/git-auto-commit-action@v4 job in https://github.com/PolarisSS13/Polaris/blob/master/.github/workflows/autochangelog.yml as

with:
  token: ${{ secrets.THE_BOT_PAT_SECRET_NAME }}

An alternative to a bot account is @Atermonera or @Woodratt generating a PAT, but that compromises branch safety somewhat, as well as commit identity separation.

If getting automatic changelogs working again is undesirable (@Cerebulon has mentioned enjoying doing it by hand) the workflow should be disabled, since it's been generating errors on every PR merge for two years.

@Spookerton Spookerton added Priority: Low [PR/Issue] This does not need short term resolution. Quality of Life [PR] This improves user experience somehow. Repo [PR/Issue] This relates to tooling, dev configuration, or dependencies. labels Sep 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Priority: Low [PR/Issue] This does not need short term resolution. Quality of Life [PR] This improves user experience somehow. Repo [PR/Issue] This relates to tooling, dev configuration, or dependencies.
Projects
None yet
Development

No branches or pull requests

3 participants