Skip to content

Conversation

@koumajos
Copy link
Contributor

@koumajos koumajos commented Feb 8, 2022

No description provided.

@codecov-commenter
Copy link

codecov-commenter commented Feb 8, 2022

Codecov Report

Merging #67 (3a7f4ee) into master (1efaeb0) will decrease coverage by 0.04%.
The diff coverage is 0.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #67      +/-   ##
==========================================
- Coverage   52.17%   52.12%   -0.05%     
==========================================
  Files          79       79              
  Lines        7249     7255       +6     
==========================================
  Hits         3782     3782              
- Misses       3467     3473       +6     
Flag Coverage Δ
tests 52.12% <0.00%> (-0.05%) ⬇️

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

Impacted Files Coverage Δ
input/parser.cpp 49.73% <0.00%> (-1.62%) ⬇️
input/pcap.cpp 28.86% <0.00%> (ø)

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 1efaeb0...3a7f4ee. Read the comment docs.

Copy link
Contributor

@cejkato2 cejkato2 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There are some coding style issues. See suggested changes to learn about it.

@koumajos koumajos closed this Feb 14, 2022
@koumajos koumajos reopened this Feb 14, 2022
koumajos and others added 4 commits February 14, 2022 10:05
Co-authored-by: Tomas Cejka <cejkat@cesnet.cz>
Co-authored-by: Tomas Cejka <cejkat@cesnet.cz>
Co-authored-by: Tomas Cejka <cejkat@cesnet.cz>
Co-authored-by: Tomas Cejka <cejkat@cesnet.cz>
@cejkato2 cejkato2 merged commit 653c9e1 into CESNET:master Feb 14, 2022
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.

3 participants