Skip to content

Add single header amalgamation facilities #172

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 2 commits into from
Dec 28, 2021
Merged

Add single header amalgamation facilities #172

merged 2 commits into from
Dec 28, 2021

Conversation

biojppm
Copy link
Owner

@biojppm biojppm commented Dec 15, 2021

No description provided.

@biojppm biojppm marked this pull request as draft December 15, 2021 23:34
@codecov
Copy link

codecov bot commented Dec 15, 2021 •
edited
Loading

Codecov Report

Merging #172 (1558853) into master (efc8b0e) will increase coverage by 0.03%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #172      +/-   ##
==========================================
+ Coverage   96.35%   96.39%   +0.03%     
==========================================
  Files          63       63              
  Lines       12824    12796      -28     
==========================================
- Hits        12357    12335      -22     
+ Misses        467      461       -6
Impacted Files Coverage Δ
src/c4/yml/common.hpp 100.00% <ø> (+9.52%) ⬆️
src/c4/yml/emit.def.hpp 96.08% <ø> (ø)
src/c4/yml/emit.hpp 100.00% <ø> (ø)
test/test_basic.cpp 99.92% <ø> (-0.01%) ⬇️
test/test_case.hpp 36.11% <ø> (ø)
samples/quickstart.cpp 99.51% <100.00%> (+<0.01%) ⬆️
src/c4/yml/common.cpp 88.09% <100.00%> (+2.67%) ⬆️
src/c4/yml/detail/stack.hpp 100.00% <100.00%> (ø)
src/c4/yml/parse.cpp 92.73% <100.00%> (ø)
src/c4/yml/parse.hpp 98.59% <100.00%> (+0.02%) ⬆️
... and 11 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 efc8b0e...1558853. Read the comment docs.

@biojppm biojppm changed the title [wip] amalgamation WIP [wip] amalgamation Dec 15, 2021
@biojppm biojppm force-pushed the amalgamate branch 2 times, most recently from 9e93180 to 0c3d843 Compare December 16, 2021 10:34
@biojppm biojppm force-pushed the amalgamate branch 7 times, most recently from 2b1d14a to 513e42b Compare December 28, 2021 00:18
@biojppm biojppm changed the title [wip] amalgamation Add amalgamation facilities Dec 28, 2021
@biojppm biojppm changed the title Add amalgamation facilities Add single header amalgamation facilities Dec 28, 2021
@biojppm biojppm marked this pull request as ready for review December 28, 2021 13:16
@biojppm biojppm merged commit 68e2029 into master Dec 28, 2021
@biojppm biojppm deleted the amalgamate branch December 28, 2021 13:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@biojppm