Skip to content

Conversation

Remi-Gau
Copy link
Contributor

No description provided.

@pep8speaks
Copy link

pep8speaks commented Jul 29, 2022

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

Line 7:1: E402 module level import not at top of file
Line 9:1: E402 module level import not at top of file

Line 130:57: E203 whitespace before ':'
Line 202:23: E203 whitespace before ':'
Line 219:36: E203 whitespace before ':'
Line 357:57: E203 whitespace before ':'

Line 1151:23: E203 whitespace before ':'
Line 1168:36: E203 whitespace before ':'
Line 1306:57: E203 whitespace before ':'
Line 1380:57: E203 whitespace before ':'

Comment last updated at 2022-07-29 14:52:14 UTC

@codecov
Copy link

codecov bot commented Jul 29, 2022

Codecov Report

Merging #733 (f0a2b71) into main (828f157) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #733   +/-   ##
=======================================
  Coverage   77.36%   77.36%           
=======================================
  Files         216      216           
  Lines        5562     5562           
=======================================
  Hits         4303     4303           
  Misses       1259     1259           
Flag Coverage Δ
unittests 77.36% <ø> (ø)

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

📣 Codecov can now indicate which changes are the most critical in Pull Requests. Learn more

@Remi-Gau Remi-Gau merged commit 7330cdc into cpp-lln-lab:main Jul 29, 2022
@Remi-Gau Remi-Gau deleted the cli branch July 29, 2022 16:38
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.

2 participants