Skip to content

Fix: Remove quotation marks in JSON report#284

Merged
pulorsok merged 2 commits into
ev-flow:masterfrom
haeter525:fix_json_report
Nov 2, 2021
Merged

Fix: Remove quotation marks in JSON report#284
pulorsok merged 2 commits into
ev-flow:masterfrom
haeter525:fix_json_report

Conversation

@haeter525

Copy link
Copy Markdown
Member

Close #283

@pep8speaks

pep8speaks commented Nov 2, 2021

Copy link
Copy Markdown

Hello @haeter525! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

Line 537:80: E501 line too long (84 > 79 characters)
Line 548:80: E501 line too long (88 > 79 characters)

Comment last updated at 2021-11-02 09:19:23 UTC

@haeter525 haeter525 assigned haeter525 and pulorsok and unassigned haeter525 Nov 2, 2021
@pulorsok pulorsok self-requested a review November 2, 2021 08:57
@haeter525 haeter525 changed the title Remove quotation marks in JSON report Fix: Remove quotation marks in JSON report Nov 2, 2021
@codecov-commenter

Copy link
Copy Markdown

Codecov Report

Merging #284 (2c96a9f) into master (5707714) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #284   +/-   ##
=======================================
  Coverage   80.01%   80.01%           
=======================================
  Files          49       49           
  Lines        3568     3568           
=======================================
  Hits         2855     2855           
  Misses        713      713           
Flag Coverage Δ
unittests 80.01% <ø> (ø)

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

Impacted Files Coverage Δ
quark/core/quark.py 70.93% <ø> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5707714...2c96a9f. Read the comment docs.

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.

Redundant quotation marks show in the JSON report

4 participants