NotificationsYou must be signed in to change notification settings
Fork673
Star2.4k

fix: wrong variable name#1332

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

Jump to bottom

Merged

nejch merged 1 commit intopython-gitlab:masterfromJohnVillalovos:jlvillal/fix_variable

Feb 25, 2021

Merged

fix: wrong variable name#1332

nejch merged 1 commit intopython-gitlab:masterfromJohnVillalovos:jlvillal/fix_variable

Feb 25, 2021

Conversation

Copy link

Member

JohnVillalovos commentedFeb 25, 2021

Discovered this when I ran flake8 on the file. Unfortunately I was the
one who introduced this wrong variable name :(

fix: wrong variable name

15ec41c

Discovered this when I ran flake8 on the file. Unfortunately I was theone who introduced this wrong variable name :(

Copy link

codecov-io commentedFeb 25, 2021

Codecov Report

Merging#1332 (c2857aa) intomaster (0b67ca2) willnot change coverage.
The diff coverage is0.00%.

@@           Coverage Diff           @@##           master    #1332   +/-   ##=======================================  Coverage   80.29%   80.29%           =======================================  Files          72       72             Lines        3746     3746           =======================================  Hits         3008     3008             Misses        738      738

Flag	Coverage Δ
unit	`80.29% <0.00%> (ø)`

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

Impacted Files	Coverage Δ
gitlab/cli.py	`53.15% <0.00%> (ø)`

Continue to review full report at Codecov.

Legend -Click here to learn more
Δ = absolute <relative> (impact),ø = not affected,? = missing data
Powered byCodecov. Last update0b67ca2...15ec41c. Read thecomment docs.