Skip to content

chore: add get() methods for GetWithoutIdMixin based classes #1710

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 11, 2021

Conversation

JohnVillalovos
Copy link
Member

Add the get() methods for the GetWithoutIdMixin based classes.

Update the tests/meta/test_ensure_type_hints.py tests to check to
ensure that the get methods are defined with the correct return type.

@JohnVillalovos JohnVillalovos force-pushed the jlvillal/get_without_id branch from 2f550e2 to d79ff6a Compare November 24, 2021 21:55
@codecov-commenter
Copy link

codecov-commenter commented Nov 24, 2021

Codecov Report

Merging #1710 (d27c50a) into main (4945353) will decrease coverage by 0.01%.
The diff coverage is 84.61%.

@@            Coverage Diff             @@
##             main    #1710      +/-   ##
==========================================
- Coverage   91.95%   91.94%   -0.02%     
==========================================
  Files          75       75              
  Lines        4689     4719      +30     
==========================================
+ Hits         4312     4339      +27     
- Misses        377      380       +3     
Flag Coverage Δ
cli_func_v4 81.24% <51.28%> (-0.16%) ⬇️
py_func_v4 80.84% <64.10%> (-0.10%) ⬇️
unit 83.13% <71.79%> (-0.03%) ⬇️

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

Impacted Files Coverage Δ
gitlab/v4/objects/push_rules.py 92.85% <0.00%> (ø)
gitlab/v4/objects/export_import.py 94.59% <50.00%> (ø)
gitlab/v4/objects/merge_request_approvals.py 90.58% <71.42%> (-1.82%) ⬇️
gitlab/v4/objects/notification_settings.py 96.42% <85.71%> (-3.58%) ⬇️
gitlab/v4/objects/appearance.py 100.00% <100.00%> (ø)
gitlab/v4/objects/pipelines.py 93.87% <100.00%> (+0.12%) ⬆️
gitlab/v4/objects/settings.py 95.45% <100.00%> (ø)
gitlab/v4/objects/statistics.py 100.00% <100.00%> (ø)
gitlab/v4/objects/users.py 95.90% <100.00%> (+0.11%) ⬆️

@JohnVillalovos JohnVillalovos force-pushed the jlvillal/get_without_id branch from d79ff6a to 2e9cbda Compare November 24, 2021 22:01
@JohnVillalovos JohnVillalovos changed the title chore: Add get() methods for GetWithoutIdMixin based classs chore: add get() methods for GetWithoutIdMixin based classs Nov 24, 2021
@JohnVillalovos JohnVillalovos changed the title chore: add get() methods for GetWithoutIdMixin based classs chore: add get() methods for GetWithoutIdMixin based classes Nov 24, 2021
@JohnVillalovos JohnVillalovos force-pushed the jlvillal/get_without_id branch from 2e9cbda to ab82fd8 Compare November 24, 2021 22:06
@JohnVillalovos JohnVillalovos requested a review from nejch November 24, 2021 22:09
@JohnVillalovos JohnVillalovos force-pushed the jlvillal/get_without_id branch 4 times, most recently from 33b2d56 to 1453601 Compare December 2, 2021 00:04
@JohnVillalovos JohnVillalovos force-pushed the jlvillal/get_without_id branch from 1453601 to 46f4d1a Compare December 6, 2021 06:09
Add the get() methods for the GetWithoutIdMixin based classes.

Update the tests/meta/test_ensure_type_hints.py tests to check to
ensure that the get methods are defined with the correct return type.
@JohnVillalovos JohnVillalovos force-pushed the jlvillal/get_without_id branch from 46f4d1a to d27c50a Compare December 8, 2021 15:36
@nejch nejch merged commit ac5defa into main Dec 11, 2021
@nejch nejch deleted the jlvillal/get_without_id branch December 11, 2021 11:07
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