solarwinds / appoptics-apm-node

AppOptics APM Instrumentation Agent for Node.js
Apache License 2.0
11 stars 9 forks source link

AO-21098-Add-Support-for-Node-18-Replacing-17 #247

Closed ronilan closed 2 years ago

ronilan commented 2 years ago

Overview

This pull request adds support or Node 18 replacing Node 17.

Changes

Changes are limited to dev setup and GitHub Actions workflows.

Issues

Tests

All acceptance tests pass. Document workflow ran as expected. Supported components data generated.

ronilan commented 2 years ago

LGTM, thanks @ronilan! Curious, is the additional check in test/versions.js required in addition to the semver check in test/probes/restify.tests.js or an optimization?

test/probes/restify.tests.js -> tests -> all day-to-day workflows test/versions.js -> testeachversion -> Document workflows -> supported components

Version check in test file makes sure test do not fail, as a result testeachversion assumes the version is good, thus must exclude version explicitly from the versions checked.

Yep. Complex...