Skip to content

Conversation

@coder137
Copy link
Owner

  • Removed *_loader and *_storer files
  • Added serialization_interface file for loading and storing serialized data into files

@coder137 coder137 mentioned this pull request Feb 10, 2022
2 tasks
@codecov
Copy link

codecov bot commented Feb 10, 2022

Codecov Report

Merging #192 (b67e7db) into main (3680ceb) will decrease coverage by 0.01%.
The diff coverage is 99.42%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #192      +/-   ##
==========================================
- Coverage   99.31%   99.31%   -0.01%     
==========================================
  Files          55       53       -2     
  Lines        1748     1733      -15     
  Branches      475      464      -11     
==========================================
- Hits         1736     1721      -15     
  Partials       12       12              
Impacted Files Coverage Δ
...arget/include/target/interface/builder_interface.h 100.00% <ø> (ø)
buildcc/lib/target/include/target/target_info.h 100.00% <ø> (ø)
buildcc/lib/target/src/target/target.cpp 100.00% <ø> (ø)
...b/target/src/generator/generator_serialization.cpp 96.15% <96.15%> (ø)
...dcc/lib/target/src/target/target_serialization.cpp 98.84% <98.84%> (ø)
buildcc/lib/target/include/target/common/path.h 100.00% <100.00%> (ø)
buildcc/lib/target/include/target/generator.h 100.00% <100.00%> (ø)
...include/target/interface/serialization_interface.h 100.00% <100.00%> (ø)
...ude/target/serialization/generator_serialization.h 100.00% <100.00%> (ø)
...nclude/target/serialization/target_serialization.h 100.00% <100.00%> (ø)
... and 17 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 3680ceb...b67e7db. Read the comment docs.

@coder137 coder137 merged commit 99bcad0 into main Feb 11, 2022
@coder137 coder137 deleted the serialization_interface branch February 11, 2022 03:05
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