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

refactor(objects): remove deprecated constants defined in objects by nejch · Pull Request #1593 · python-gitlab/python-gitlab · GitHub

Repository navigation

refactor(objects): remove deprecated constants defined in objects - #1593

Merged
JohnVillalovos merged 1 commit into
masterfrom
refactor/consts-in-const-module
Sep 11, 2021
Merged

JohnVillalovos merged 1 commit into
masterfrom
refactor/consts-in-const-module

Conversation

nejch commented Sep 8, 2021 •
edited
Loading

Copy link
Copy Markdown
Member

BREAKING CHANGE: remove deprecated constants defined in gitlab.v4.objects, and use only gitlab.const module

FYI: the visibility as int constants can safely be changed to string - they were only used in GitLab API v3 prior to gitlab v9.0. https://gitlab.com/gitlab-org/gitlab-foss/-/merge_requests/9337

nejch changed the title refactor(const): remove deprecated constants defined in objects refactor(objects): remove deprecated constants defined in objects Sep 8, 2021
nejch mentioned this pull request Sep 8, 2021
15 tasks done

codecov-commenter commented Sep 8, 2021 •
edited
Loading

Copy link
Copy Markdown

Codecov Report

Merging #1593 (b8d379d) into master (2b8a94a) will decrease coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #1593      +/-   ##
==========================================
- Coverage   91.59%   91.57%   -0.02%     
==========================================
  Files          74       74              
  Lines        4272     4263       -9     
==========================================
- Hits         3913     3904       -9     
  Misses        359      359              
Flag Coverage Δ
cli_func_v4 81.56% <100.00%> (-0.04%) ⬇️
py_func_v4 80.57% <100.00%> (-0.05%) ⬇️
unit 83.34% <100.00%> (-0.04%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
gitlab/v4/objects/__init__.py 100.00% <ø> (ø)
gitlab/const.py 100.00% <100.00%> (ø)

nejch marked this pull request as draft September 8, 2021 22:12
BREAKING CHANGE: remove deprecated constants defined in
gitlab.v4.objects, and use only gitlab.const module
nejch force-pushed the refactor/consts-in-const-module branch from ce39754 to b8d379d Compare September 11, 2021 10:03
nejch marked this pull request as ready for review September 11, 2021 10:13
JohnVillalovos merged commit 3f320af into master Sep 11, 2021
JohnVillalovos deleted the refactor/consts-in-const-module branch September 11, 2021 14:14
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.

3 participants


Back | FazBrowse Home | New Git URL