-
Notifications
You must be signed in to change notification settings - Fork 105
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
added update branch
before auto-merge
#429
Conversation
Updated `auto-merge-bot` to version `1.1.0` which has the option to update a branch to be up-to-date before merging.
with: | ||
GITHUB_TOKEN: ${{ steps.merge_token.outputs.token }} | ||
MERGE_METHOD: "SQUASH" | ||
ALLOWLIST: ${{ steps.fellows.outputs.github-handles }} | ||
UPDATE_BEFORE_MERGE: true |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Otherwise the "automatic merge" it not triggered any way.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I mean, it was doing this before already or not?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It wasn't doing this before.
The new version runs the updatePullRequestBranch
mutation before enabling auto-merge if this field is enabled.
Apologizes for the delay of the feature, but now it should work as intended and always update the PR before enabling auto-merge.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice!
/merge |
Enabled Available commands
For more information see the documentation |
9cfa42c
into
polkadot-fellows:main
Updated
auto-merge-bot
to version1.1.0
which has the option to update a branch to be up-to-date before merging.