-
Notifications
You must be signed in to change notification settings - Fork 0
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
🎨 Create dependabot automerge action #187
Conversation
I tested the workflow in my own testing environment: |
But your checks in the repository are not required. I had the experience when you have required checks that it can fail. Edit: And you don't have CODEOWNERS reviews enabled, right? |
I had a required step and it works perfectly. 🙌 The codeowner issue can be solved in each repository explicitly with an adaption of the CODEOWNERS file. But I would keep this configuration in the specific repository. 🎉 And if we find that we need to adjust something, then I would improve the action step by step. |
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 agree that we can iterate on it if it is in a basic working state, which it seems to be. I would like see a bit of additional docs (e.g. CODEOWNERS). Maybe add a verbal hint, that this workflow will be called by the on: pull_request
workflow, thereby on dependabot PRs.
dd97719
to
167f0b7
Compare
Type of Change
Description
Checklist