apiaryio / drafter

API Blueprint Parser (C++)
https://apiblueprint.org/
MIT License
301 stars 54 forks source link

Release 5.1 - Support current compilers, nodejs 16+ support #794

Closed kylef closed 1 year ago

kylef commented 1 year ago

What:

This change releases Drafter 5.1 (Drafter, the API Blueprint Parser written in C/C++). This release contains existing bug fixes that had been merged a few years back and never got released. Along with numerous changes for CI to currently pass, and for the project to build in modern systems, and Node JS 16.

Both Catch (test library dev dependency) and Boost are upgraded. Boost is vendored in, this is done in its own commit to make review easier as it is a "large" change to boost source. Catch2 older version no longer allows project to be built on current compilers and faced similar issue.

Why:

Overall, fighting bit rot.

Criticality:

Medium

Blast radius:

Low - No change to API or ABI Compatibility. Unexpected to change any minimum compiler requirements of the project.

Expected outcome:

Not a lot changes.

Potential impact and visibility:

No expected impact.

Test results:

All existing test coverage passes (unit, integration).

I am unable to test the project still builds on windows. Appveyor (our windows CI) doesn't seem to be anything that's still being paid for. It's unlikely that Windows incompatibilities have arisen given only functional change is to boost (which they should have sufficient tests for).

Screenshot 2023-05-18 at 00 50 31