Skip to content

Conversation

@Remi-Gau
Copy link
Contributor

No description provided.

@pep8speaks
Copy link

pep8speaks commented May 17, 2022

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

There are currently no PEP 8 issues detected in this Pull Request. Cheers! 🍻

Comment last updated at 2022-05-17 15:45:56 UTC

@codecov
Copy link

codecov bot commented May 17, 2022

Codecov Report

Merging #573 (303f868) into dev (f8da99c) will increase coverage by 0.10%.
The diff coverage is 85.71%.

@@            Coverage Diff             @@
##              dev     #573      +/-   ##
==========================================
+ Coverage   72.87%   72.97%   +0.10%     
==========================================
  Files         186      186              
  Lines        4420     4426       +6     
==========================================
+ Hits         3221     3230       +9     
+ Misses       1199     1196       -3     
Flag Coverage Δ
unittests 72.97% <85.71%> (+0.10%) ⬆️

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

Impacted Files Coverage Δ
src/messages/printCredits.m 94.59% <85.71%> (-2.18%) ⬇️
src/infra/returnRootDir.m 30.76% <0.00%> (+30.76%) ⬆️

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

@Remi-Gau Remi-Gau merged commit 5f5c141 into cpp-lln-lab:dev May 17, 2022
@Remi-Gau Remi-Gau deleted the infra branch May 17, 2022 16:00
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