Skip to content

fix bug in labelling in legend#957

Merged
fso42 merged 1 commit intomasterfrom
fixBugFig
Jan 18, 2024
Merged

fix bug in labelling in legend#957
fso42 merged 1 commit intomasterfrom
fixBugFig

Conversation

@awirb
Copy link
Copy Markdown
Contributor

@awirb awirb commented Jan 18, 2024

No description provided.

@awirb awirb added the confirmed Something isn't working label Jan 18, 2024
@awirb awirb requested a review from fso42 January 18, 2024 15:14
@pep8speaks
Copy link
Copy Markdown

Hello @awirb! Thanks for opening this PR.

Line 233:13: E231 missing whitespace after ','
Line 232:13: E231 missing whitespace after ','
Line 231:13: E231 missing whitespace after ','
Line 230:13: E231 missing whitespace after ','
Line 229:13: E231 missing whitespace after ','
Line 228:13: E231 missing whitespace after ','
Line 227:13: E231 missing whitespace after ','
Line 226:13: E231 missing whitespace after ','
Line 225:13: E231 missing whitespace after ','
Line 224:13: E231 missing whitespace after ','
Line 223:13: E231 missing whitespace after ','
Line 222:13: E231 missing whitespace after ','
Line 221:13: E231 missing whitespace after ','
Line 220:13: E231 missing whitespace after ','
Line 219:13: E231 missing whitespace after ','
Line 218:13: E231 missing whitespace after ','
Line 217:13: E231 missing whitespace after ','
Line 216:13: E231 missing whitespace after ','
Line 164:48: E712 comparison to False should be 'if cond is False:' or 'if not cond:'
Line 106:121: E501 line too long (121 > 120 characters)

Do see the Hitchhiker's guide to code style

@qlty-cloud-legacy
Copy link
Copy Markdown

Code Climate has analyzed commit 363d010 and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 100.0% (50% is the threshold).

This pull request will bring the total coverage in the repository to 73.9% (0.0% change).

View more on Code Climate.

@codecov
Copy link
Copy Markdown

codecov bot commented Jan 18, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (3fbde2d) 73.92% compared to head (363d010) 73.92%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #957   +/-   ##
=======================================
  Coverage   73.92%   73.92%           
=======================================
  Files          64       64           
  Lines       14675    14675           
=======================================
  Hits        10848    10848           
  Misses       3827     3827           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@fso42 fso42 merged commit 3dcd040 into master Jan 18, 2024
@fso42 fso42 deleted the fixBugFig branch January 18, 2024 16:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

confirmed Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants