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

auto-merge: replaced review bot for merge bot #98

Merged

Conversation

Bullrich
Copy link
Contributor

When adapted the code from #88 I forgot to update the location of the credentials, so it was using review-bot which can only enable auto-merge but can not merge a ready to merge PR. Follow up to #97

When adapted the code from polkadot-fellows#88 I forgot to update the location of the credentials, so it was using review-bot which can only enable auto-merge but can not merge a successful PR
@Bullrich Bullrich requested review from andresilva and bkchr November 21, 2023 08:18
@Bullrich
Copy link
Contributor Author

/merge

Copy link

There was a problem running the action.

❌😵❌

Please find more information in the logs.

@Bullrich Bullrich enabled auto-merge (squash) November 21, 2023 08:19
@Bullrich
Copy link
Contributor Author

only enable auto-merge but can not merge a ready to merge PR

Moments later:

There was a problem running the action.

❌😵❌

It seems I was wrong about review-bot 🤔

@Bullrich Bullrich merged commit 41d3ca6 into polkadot-fellows:main Nov 22, 2023
9 checks passed
@Bullrich Bullrich deleted the bullrich/auto-merge-bot-migration branch November 22, 2023 11:15
Sophia-Gold pushed a commit to Sophia-Gold/polkadot-runtimes that referenced this pull request Nov 22, 2023
When adapted the code from polkadot-fellows#88 I forgot to update the location of the
credentials, so it was using review-bot which can only enable auto-merge
but can not merge a ready to merge PR. Follow up to polkadot-fellows#97
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