search

fktn-k / fkYAML

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

#180 clean up public apis #183

Closed fktn-k closed 11 months ago

fktn-k commented 11 months ago

Along with developing the fkYAML library, some public APIs have become unnecessarily public.
Taht is because more convenient APIs are introduced, such as basic_node constructor with std::initializer_list has covered factory methods of basic_node objects.
Also, deserialization API overload has been removed which could possibly cause undefined behaviors.
So, this PR has removed those APIs so that the fkYAML library will have more readable & understandable APIs.

coveralls commented 11 months ago

Coverage Status

coverage: 100.0%. remained the same when pulling aff3715098e2a43fcbdcb857c017b4d0c2c0d143 on feature/180_clean_up_public_apis into b116abb4c437ea7e6ad375d7d25bbcdc1dc16e16 on develop.