Attempt to fix Dependabot auto-merge workflow #6975
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
westonruter
added
the
Infrastructure
|
Plugin builds for 3dde1c4 are ready 🛎️!
|
|
Another issue that references this error: dependabot/fetch-metadata#172 |
|
Welp, it didn't even trigger with this (e.g. #6977) |
|
Somehow the auto-merge checkbox was not checked: I've just enabled it. Let's see if that changes things. |
westonruter
added
the
Changelogged
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
This follows up on #6964 (comment).
Based on cli/cli#3660 (comment) (h/t @pierlon) it seems running
gh merge --autoit seems that theDependabot auto-mergeworkflow may be failing due to one of the preconditions not being met. Namely, the required checks being run in theBuild, test & measureworkflow. Therefore, this PR makes it so that theDependabot auto-mergeworkflow will only be triggered once the aforementioned workflow is completed. Additionally, the approval of the PR is now done after the enabling of auto-merge as apparently enabling auto-merge can't be done if it could be merged already per cli/cli#3660 (reply in thread). So there seems to be two competing conditions, one for the required checks and another for the PR approval. Maybe if auto-merge is enabled in between them then it will succeed.This PR also amends #6964 to enable auto-merge for
minorupdates as well aspatchupdates.Checklist