Skip to content
This repository has been archived by the owner on Jan 26, 2018. It is now read-only.

Overrides existing protected branch settings #29

Closed
kphelps opened this issue Jul 7, 2016 · 2 comments
Closed

Overrides existing protected branch settings #29

kphelps opened this issue Jul 7, 2016 · 2 comments

Comments

@kphelps
Copy link

kphelps commented Jul 7, 2016

Before enabling LGTM, I had "Include Administrators" enabled and other status checks enabled on my protected branch. After enabling LGTM on the repo, "Include Administrators" was disabled and my other status check was disabled, leaving only approvals/lgtm enabled.

@bradrydzewski
Copy link
Member

tracking under #10

@bradrydzewski
Copy link
Member

bradrydzewski commented Jul 7, 2016

because the github protected branches API is unstable and in developer preview, the API behavior has changed significantly since last November. Previously the PATCH didn't replace or overwrite existing information. The API endpoint has now been completely replaced as of last week [1] and LGTM will need to be adjusted accordingly

[1] https://developer.github.com/changes/2016-06-27-protected-branches-api-update/

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

No branches or pull requests

2 participants