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

use github app for auto-merge-bot #94

Merged
merged 3 commits into from
Nov 17, 2023

Conversation

Bullrich
Copy link
Contributor

This changes the credentials for auto-merge-bot

With this it stops using a GitHub action but instead it uses a GitHub app, by using a GitHub app, PRs merged by it will trigger GitHub actions.

This is to circumvent a security measure set up by GitHub to stop recursive action executions.

This supersedes #88

This changes the credentials for auto-merge-bot

With this it stops using a GitHub action but instead it uses a GitHub app, by using a GitHub app, PRs merged by it will trigger GitHub actions.

This is to circumvent a security measure set up by GitHub to stop recursive action executions.
@Bullrich Bullrich self-assigned this Nov 16, 2023
@bkchr
Copy link
Contributor

bkchr commented Nov 17, 2023

/merge

Copy link

There was a problem running the action.

❌😵❌

Please find more information in the logs.

@Bullrich Bullrich enabled auto-merge (squash) November 17, 2023 13:43
@Bullrich Bullrich merged commit 2c6b1bf into polkadot-fellows:main Nov 17, 2023
5 checks passed
@Bullrich Bullrich mentioned this pull request Nov 20, 2023
Bullrich added a commit that referenced this pull request Nov 20, 2023
I confused `env` which is used for environment variables for
`environment` which is the key word to fetch environment secrets when
doing #94.

This will fix that mistake.
@Bullrich Bullrich deleted the auto-merge-bot-to-app branch November 21, 2023 08:14
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