FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

refactor(objects): remove deprecated branch protect methods by nejch · Pull Request #1598 · python-gitlab/python-gitlab · GitHub

Repository navigation

refactor(objects): remove deprecated branch protect methods - #1598

Merged
JohnVillalovos merged 1 commit into
masterfrom
refactor/remove-deprecated-branch-protect
Oct 6, 2021
Merged

JohnVillalovos merged 1 commit into
masterfrom
refactor/remove-deprecated-branch-protect

Conversation

nejch commented Sep 11, 2021

Copy link
Copy Markdown
Member

BREAKING CHANGE: remove deprecated branch protect methods in favor of
the more complete Protected branches API.

Closes #821.

@max-wittig WDYT about removing this completely? Then people will stop getting confused about the two APIs like in #1355 and #821. The endpoint still technically works, but the GitLab API docs removed it ages ago and they point to the dedicated protected branches API now (https://docs.gitlab.com/ee/api/branches.html#protect-repository-branch).

Last time we kept a deprecated endpoint (release via tags API) it bit us because GitLab then "silently" removed it. Just adding this to the list of breaking changes as the next scheduled job will trigger 3.0.0.

BREAKING CHANGE: remove deprecated branch protect methods in favor of
the more complete protected branches API.
nejch mentioned this pull request Sep 11, 2021
15 tasks done
nejch assigned JohnVillalovos and unassigned max-wittig Oct 6, 2021
JohnVillalovos enabled auto-merge (rebase) October 6, 2021 19:08
JohnVillalovos merged commit 9656a16 into master Oct 6, 2021
JohnVillalovos deleted the refactor/remove-deprecated-branch-protect branch October 6, 2021 19:09
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Additional branch.protect() fields

3 participants


Back | FazBrowse Home | New Git URL