Skip to content

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

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

Merged
merged 1 commit into from
Sep 11, 2021

Conversation

nejch
Copy link
Member

@nejch nejch commented Sep 8, 2021

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 nejch changed the title refactor(const): remove deprecated constants defined in objects refactor(objects): remove deprecated constants defined in objects Sep 8, 2021
@nejch nejch mentioned this pull request Sep 8, 2021
15 tasks
@codecov-commenter
Copy link

codecov-commenter commented Sep 8, 2021

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 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 nejch force-pushed the refactor/consts-in-const-module branch from ce39754 to b8d379d Compare September 11, 2021 10:03
@nejch nejch marked this pull request as ready for review September 11, 2021 10:13
@JohnVillalovos JohnVillalovos merged commit 3f320af into master Sep 11, 2021
@JohnVillalovos JohnVillalovos deleted the refactor/consts-in-const-module branch September 11, 2021 14:14
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