Skip to content

feat(api): add project label promotion #1610

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Oct 20, 2021
Merged

feat(api): add project label promotion #1610

merged 1 commit into from
Oct 20, 2021

Conversation

StingRayZA
Copy link
Contributor

Adds a mixin that allows the /promote endpoint to be called on project labels.

Fixes #1602

@codecov-commenter
Copy link

codecov-commenter commented Sep 27, 2021 •
edited
Loading

Codecov Report

Merging #1610 (c26b619) into master (a5d8b7f) will increase coverage by 0.03%.
The diff coverage is 95.65%.

@@            Coverage Diff             @@
##           master    #1610      +/-   ##
==========================================
+ Coverage   91.57%   91.61%   +0.03%     
==========================================
  Files          74       74              
  Lines        4263     4294      +31     
==========================================
+ Hits         3904     3934      +30     
- Misses        359      360       +1
Flag Coverage Δ
cli_func_v4 81.50% <65.21%> (-0.06%) ⬇️
py_func_v4 80.71% <95.65%> (+0.14%) ⬆️
unit 83.27% <65.21%> (-0.07%) ⬇️

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

Impacted Files Coverage Δ
gitlab/mixins.py 91.31% <95.00%> (+0.19%) ⬆️
gitlab/exceptions.py 99.29% <100.00%> (+0.01%) ⬆️
gitlab/v4/objects/labels.py 93.47% <100.00%> (ø)
gitlab/v4/objects/personal_access_tokens.py 100.00% <0.00%> (ø)
gitlab/v4/objects/users.py 98.40% <0.00%> (+0.01%) ⬆️

@JohnVillalovos
Copy link
Member

Thanks for this. This is going to need some docs updates too, so that people know it is there and also how to use it.

@StingRayZA
Copy link
Contributor Author

Sure - will add them now the CI has given the code a 👍

@StingRayZA
Copy link
Contributor Author

I've removed the CLI action (It wasn't testing well, and I don't think I know how to make it work without breaking backwards compatibility)

Also added a note into the docs...

nejch
nejch requested changes Oct 6, 2021
View reviewed changes
Copy link
Member

@nejch nejch left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you @StingRayZA! I just had a few questions.

I'm unsure if /promote is an endpoint that will appear elsewhere and warrant its own Mixin, but it looks clean this way :)

@StingRayZA
Copy link
Contributor Author

Thanks for the review @nejch - I've got a tweak or two to make and I'll post them ASAP...

@StingRayZA StingRayZA requested a review from nejch October 11, 2021 13:26
@nejch nejch assigned nejch and unassigned StingRayZA Oct 14, 2021
nejch
nejch requested changes Oct 17, 2021
View reviewed changes
@nejch nejch assigned StingRayZA and unassigned nejch Oct 17, 2021
@nejch
Copy link
Member

nejch commented Oct 17, 2021

Thanks again @StingRayZA. Just a nitpick for tests and if you could maybe rebase on top of main? :)

@StingRayZA
feat(api): add project label promotion
6d7c88a 
Adds a mixin that allows the /promote endpoint to be called.

Signed-off-by: Raimund Hook <raimund.hook@exfo.com>
@StingRayZA
Copy link
Contributor Author

All done @nejch - updated and rebased 😁

@StingRayZA StingRayZA requested a review from nejch October 18, 2021 09:34
@nejch nejch merged commit 853d850 into python-gitlab:main Oct 20, 2021
@nejch
Copy link
Member

nejch commented Oct 20, 2021

Thanks for the contribution @StingRayZA! :) Looking forward to the milestone PR 😁

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nejch nejch Awaiting requested review from nejch

Assignees

@StingRayZA StingRayZA

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Feature Request: Support promoting project labels
4 participants
@StingRayZA @codecov-commenter @JohnVillalovos @nejch