Skip to content

fix: Use the ProjectIterationManager within the Project object #2404

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
Dec 3, 2022

Conversation

hollerdawg
Copy link

The Project object was previously using the GroupIterationManager resulting in the incorrect API endpoint being used. Utilize the correct ProjectIterationManager instead.

Resolves #2403

The Project object was previously using the GroupIterationManager
resulting in the incorrect API endpoint being used. Utilize the correct
ProjectIterationManager instead.

Resolves python-gitlab#2403
@codecov-commenter
Copy link

Codecov Report

Merging #2404 (1e63822) into main (22b11db) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main    #2404   +/-   ##
=======================================
  Coverage   95.97%   95.97%           
=======================================
  Files          80       80           
  Lines        5342     5342           
=======================================
  Hits         5127     5127           
  Misses        215      215           
Flag Coverage Δ
api_func_v4 83.73% <100.00%> (ø)
cli_func_v4 82.38% <100.00%> (ø)
unit 87.75% <100.00%> (ø)

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

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

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.

Thanks for the catch @hollerdawg!

Note to self, this was probably an oversight when I added it initially because "project iterations" don't really exist anymore and associated group iterations are returned instead. So we might need to revisit this a bit if there's any kind of iteration-level endpoint later.

@nejch nejch merged commit 44f05dc into python-gitlab:main Dec 3, 2022
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.

project iterations are broken
3 participants