Skip to content

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.

4 participants