-
Notifications
You must be signed in to change notification settings - Fork 4
Add auto-approve step for go code by dependabot #65
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
Conversation
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 wanna make sure I. understand what this is doing... if depndabot submits a PR this workflow will trigger. in the workflow, it'll check out thecode. setup go and run go mod tidy then check it back in?
So this will have a tidy version of the module version changes that dependabot suggests?
Did y'all run into any issues auto approving/merging these kinds. of changes?
I'm just curious since I didn't see a change on the dependabot config limiting this to minor/patch changes oh and automerge config.
Yeah, it essentially does some cleanup which dependabot doesn't do. And it should auto-merge if everything goes right, meaning no humans need to be involved. I really want to do some more testing with this as well but it may requires some updates. The impetus for this is that the repo had two outstanding dependabot changes that hadn't been handled. All it needs is an auto-approve step and then it would do this work for us without our interference. The only problems MilMove ever had was when a library got modified that caused client side integration tests to fail. For unit testing it seemed to be fine. Plus, this has a much smaller surface area than MilMove so I'm pretty confident it will work. |
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.
Cool thank you for confirming my understanding. You definitely need the change to .dependabot/config.yml then. It doesn't have any automerge stanzas. Just add that and I'll give it the 👍
Yeah, can't believe I missed that. It's in now. |
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.
🥳
This is copied from milmove. You can see the original code here: https://github.com/transcom/mymove/blob/master/.github/workflows/go-auto-approve.yml
The point is to allow dependabot to suggest and auto-merge changes.