Skip to content

chore: change _repr_attr for Project to be path_with_namespace #2203

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
Jul 29, 2022

Conversation

JohnVillalovos
Copy link
Member

Previously _repr_attr was path but that only gives the basename of
the path. So https://gitlab.com/gitlab-org/gitlab would only show
"gitlab". Using path_with_namespace it will now show
"gitlab-org/gitlab"

Previously `_repr_attr` was `path` but that only gives the basename of
the path.  So https://gitlab.com/gitlab-org/gitlab would only show
"gitlab". Using `path_with_namespace` it will now show
"gitlab-org/gitlab"
@codecov-commenter
Copy link

Codecov Report

Merging #2203 (7cccefe) into main (17414f7) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main    #2203   +/-   ##
=======================================
  Coverage   95.46%   95.46%           
=======================================
  Files          81       81           
  Lines        5353     5353           
=======================================
  Hits         5110     5110           
  Misses        243      243           
Flag Coverage Δ
api_func_v4 81.44% <100.00%> (ø)
cli_func_v4 83.13% <100.00%> (+0.11%) ⬆️
unit 87.27% <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 100.00% <100.00%> (ø)

@JohnVillalovos JohnVillalovos requested a review from nejch July 29, 2022 17:40
@nejch nejch merged commit 98bdb98 into main Jul 29, 2022
@nejch nejch deleted the jlvillal/project_repr branch July 29, 2022 21:16
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