Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ci: simplify build system #44

Merged
merged 15 commits into from
Jul 20, 2022
Merged

ci: simplify build system #44

merged 15 commits into from
Jul 20, 2022

Conversation

maxmouchet
Copy link
Member

@maxmouchet maxmouchet commented Jun 24, 2022

@codecov
Copy link

codecov bot commented Jun 24, 2022

Codecov Report

Merging #44 (d380c6b) into main (9e4251c) will decrease coverage by 0.36%.
The diff coverage is n/a.

❗ Current head d380c6b differs from pull request most recent head e245fc8. Consider uploading reports for the commit e245fc8 to get more accurate results

@@            Coverage Diff             @@
##             main      #44      +/-   ##
==========================================
- Coverage   79.84%   79.47%   -0.37%     
==========================================
  Files          22       25       +3     
  Lines        1146     1291     +145     
  Branches      345      709     +364     
==========================================
+ Hits          915     1026     +111     
- Misses        160      165       +5     
- Partials       71      100      +29     
Impacted Files Coverage Δ
include/caracal/checked.hpp 84.21% <0.00%> (-9.54%) ⬇️
src/experimental.cpp 75.38% <0.00%> (-3.19%) ⬇️
src/reply.cpp 16.90% <0.00%> (-1.97%) ⬇️
src/sniffer.cpp 64.78% <0.00%> (-1.35%) ⬇️
src/builder.cpp 90.29% <0.00%> (-0.24%) ⬇️
src/rate_limiter.cpp 75.92% <0.00%> (-0.08%) ⬇️
src/pretty.cpp 100.00% <0.00%> (ø)
src/checksum.cpp 100.00% <0.00%> (ø)
src/timestamp.cpp 100.00% <0.00%> (ø)
include/caracal/parser.hpp 100.00% <0.00%> (ø)
... and 14 more

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 25dd65b...e245fc8. Read the comment docs.

@maxmouchet maxmouchet changed the title draft CI fixes Jun 24, 2022
@maxmouchet maxmouchet changed the title CI fixes ci: simplify build system Jul 20, 2022
@maxmouchet maxmouchet merged commit 74be77c into main Jul 20, 2022
@maxmouchet maxmouchet deleted the manylinux_2_28 branch July 20, 2022 11:06
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.

1 participant