fktn-k / fkYAML

A C++ header-only YAML library
MIT License
69 stars 7 forks source link

Fix Valgrind error in the test suite #307

Closed fktn-k closed 6 months ago

fktn-k commented 6 months ago

This PR has fixed errors reported by Valgrind.
The errors were due to missing std::fclose() calls after opening a test data file in the input_adapter class test cases.
So no changes have been made to the library source codes.


Pull Request Checklist

Read the CONTRIBUTING.md file for detailed information.

Please don't

github-actions[bot] commented 6 months ago

:octocat: Upload Coverage Event Notification

Coverage data has been uploaded for the commit 7988d1c8dc28cd098d4e46ccd9fd8032a83b9a5d. You can download the artifact which contains the same file uploaded to the Coveralls and its HTML version.

Name fkYAML_coverage.pr307.zip
ID 1398333619
URL https://github.com/fktn-k/fkYAML/actions/runs/8618357114/artifacts/1398333619
coveralls commented 6 months ago

Pull Request Test Coverage Report for Build 8618357114

Details


Totals Coverage Status
Change from base Build 8602833627: 0.0%
Covered Lines: 3464
Relevant Lines: 3464

💛 - Coveralls