-
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
Add AllowUpdateBranch to the Repository struct #2313
Add AllowUpdateBranch to the Repository struct #2313
Conversation
Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). For more information, open the CLA check for this pull request. |
Thank you, @MarkRosemaker ! |
Thank you for the suggestion, @gmlewis ! Done. :) |
Codecov Report
@@ Coverage Diff @@
## master #2313 +/- ##
=======================================
Coverage 97.84% 97.84%
=======================================
Files 115 115
Lines 10415 10416 +1
=======================================
+ Hits 10191 10192 +1
Misses 156 156
Partials 68 68
Continue to review full report at Codecov.
|
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, @MarkRosemaker !
LGTM.
Merging.
I noticed that I can edit all the settings for a repository via the API except the setting:
This PR aims to change that. I verified the changed code by making an edit request to one of my repositories.
This PR will solve #2312.