Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
nejch merged 1 commit intomainfromjlvillal/get_without_id
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.

@codecov-commenter
Copy link

codecov-commenter commentedNov 24, 2021
edited
Loading

Codecov Report

Merging#1710 (d27c50a) intomain (4945353) willdecrease coverage by0.01%.
The diff coverage is84.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
FlagCoverage Δ
cli_func_v481.24% <51.28%> (-0.16%)⬇️
py_func_v480.84% <64.10%> (-0.10%)⬇️
unit83.13% <71.79%> (-0.03%)⬇️

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

Impacted FilesCoverage Δ
gitlab/v4/objects/push_rules.py92.85% <0.00%> (ø)
gitlab/v4/objects/export_import.py94.59% <50.00%> (ø)
gitlab/v4/objects/merge_request_approvals.py90.58% <71.42%> (-1.82%)⬇️
gitlab/v4/objects/notification_settings.py96.42% <85.71%> (-3.58%)⬇️
gitlab/v4/objects/appearance.py100.00% <100.00%> (ø)
gitlab/v4/objects/pipelines.py93.87% <100.00%> (+0.12%)⬆️
gitlab/v4/objects/settings.py95.45% <100.00%> (ø)
gitlab/v4/objects/statistics.py100.00% <100.00%> (ø)
gitlab/v4/objects/users.py95.90% <100.00%> (+0.11%)⬆️

@JohnVillalovosJohnVillalovos changed the titlechore: Add get() methods for GetWithoutIdMixin based classschore: add get() methods for GetWithoutIdMixin based classsNov 24, 2021
@JohnVillalovosJohnVillalovos changed the titlechore: add get() methods for GetWithoutIdMixin based classschore: add get() methods for GetWithoutIdMixin based classesNov 24, 2021
@JohnVillalovosJohnVillalovosforce-pushed thejlvillal/get_without_id branch 4 times, most recently from33b2d56 to1453601CompareDecember 2, 2021 00:04
Add the get() methods for the GetWithoutIdMixin based classes.Update the tests/meta/test_ensure_type_hints.py tests to check toensure that the get methods are defined with the correct return type.
@nejchnejch merged commitac5defa intomainDec 11, 2021
@nejchnejch deleted the jlvillal/get_without_id branchDecember 11, 2021 11:07
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@nejchnejchnejch left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@JohnVillalovos@codecov-commenter@nejch

[8]ページ先頭

©2009-2025 Movatter.jp