-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Introduce the auto merge property #2074
Introduce the auto merge property #2074
Conversation
Hi @Richard-de-Vries, did you remember to run |
Hello @gmlewis I'm fairly new to go 😅. |
Codecov Report
@@ Coverage Diff @@
## master #2074 +/- ##
=======================================
Coverage 97.75% 97.75%
=======================================
Files 107 107
Lines 9600 9601 +1
=======================================
+ Hits 9384 9385 +1
Misses 150 150
Partials 66 66
Continue to review full report at Codecov.
|
Welcome to Go, @Richard-de-Vries ! |
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.
Thank you, @Richard-de-Vries !
LGTM.
Merging.
#2073 Add auto merge support
Pr to add the allow_auto_merge property that is newly exposed in the github api
Changed enities
Tests
I could not find what tests should be changed when adding a new property so i did not add any. If this pr should include tests please point me to the files i should be looking at ?