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

Try to fix auto-merge for dependatbot PR's #866

Merged
merged 1 commit into from
Nov 21, 2021

Conversation

DanielBadura
Copy link
Contributor

As far as I understand to docs from the auto-merge action (https://github.com/ridedott/merge-me-action#public-repositories) using the check_suite option you need to remove the if part from the action.

Copy link
Member

@Ocramius Ocramius left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's try it 🤷‍♀️

@Ocramius Ocramius added the bug label Nov 21, 2021
@Ocramius Ocramius added this to the 5.0.0 milestone Nov 21, 2021
@Ocramius Ocramius self-assigned this Nov 21, 2021
@DanielBadura
Copy link
Contributor Author

This whole auto-merge with new dependabot is such a documentation mess 😂

@Ocramius Ocramius merged commit 82f4a7b into Roave:5.0.x Nov 21, 2021
@Ocramius
Copy link
Member

Agreed, and so is GitHub actions itself.

Perhaps the only way is to "ship it yourself" with a single massive CI workflow.

@DanielBadura DanielBadura deleted the patch-1 branch November 21, 2021 16:00
@kukulich
Copy link
Collaborator

I think it will not fix it.

The action is not launched at all: https://github.com/Roave/BetterReflection/actions/workflows/merge-dependabot-upgrades.yml

@DanielBadura
Copy link
Contributor Author

Well CI is still running needs some time for infection check

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants