Skip to content

Conversation

@awirb
Copy link
Contributor

@awirb awirb commented Sep 13, 2022

No description provided.

@awirb awirb requested review from fso42 and matthiasto September 13, 2022 12:44
@pep8speaks
Copy link

Hello @awirb! Thanks for opening this PR.

Line 491:19: E225 missing whitespace around operator
Line 407:42: E241 multiple spaces after ','
Line 396:70: E251 unexpected spaces around keyword / parameter equals
Line 396:68: E251 unexpected spaces around keyword / parameter equals
Line 396:48: E251 unexpected spaces around keyword / parameter equals
Line 396:46: E251 unexpected spaces around keyword / parameter equals
Line 310:121: E501 line too long (151 > 120 characters)

Do see the Hitchhiker's guide to code style

@qlty-cloud-legacy
Copy link

Code Climate has analyzed commit 640631f and detected 0 issues on this pull request.

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

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

View more on Code Climate.

@codecov
Copy link

codecov bot commented Sep 13, 2022

Codecov Report

Merging #770 (640631f) into master (c01c010) will not change coverage.
The diff coverage is 0.00%.

@@           Coverage Diff           @@
##           master     #770   +/-   ##
=======================================
  Coverage   74.80%   74.80%           
=======================================
  Files          57       57           
  Lines       12034    12034           
=======================================
  Hits         9002     9002           
  Misses       3032     3032           
Impacted Files Coverage Δ
avaframe/out3Plot/statsPlots.py 21.48% <0.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@fso42 fso42 added the enhancement New feature or request label Sep 14, 2022
@fso42 fso42 added this to the Version 1.3 milestone Sep 14, 2022
@fso42 fso42 merged commit 38d1e8f into master Sep 14, 2022
@fso42 fso42 deleted the testPlots branch September 14, 2022 05:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants