RobotWebTools / rclnodejs / 24069374479

85%

push

github

Gate C++ Standard By ROS Distro (#1477)

- build: switch native addon C++ standard selection from Node.js-major gating to ROS-distro gating
  - remove `node_major_version` from `binding.gyp`
  - use C++17 for `ros_version <= 2505`
  - use C++20 for `ros_version > 2505`
  - apply the same rule on both Linux and Windows build paths

- docs(build): document source-build C++ toolchain requirements
  - add a `C++ Toolchain Requirements` section to `docs/BUILDING.md`
  - state that Humble, Jazzy, and Kilted build with C++17
  - state that post-Kilted distros, including Lyrical and newer Rolling snapshots, build with C++20

**Reference**:
- https://discourse.openrobotics.org/t/ros-2-lyrical-c-version/52551

Fix: #1458

1565 of 1992 branches covered (78.56%)

Branch coverage included in aggregate %.

3177 of 3560 relevant lines covered (89.24%)

449.32 hits per line