Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 95.01%. Comparing base (91e0d23) to head (0bc8c70).
⚠️ Report is 2 commits behind head on main.

@@           Coverage Diff           @@
##             main    #6019   +/-   ##
=======================================
  Coverage   95.01%   95.01%           
=======================================
  Files         313      313           
  Lines        8799     8799           
  Branches     1888     1888           
=======================================
  Hits         8360     8360           
  Misses        439      439           

The README.md claims that "Only Node.js Active or Maintenance LTS versions are supported", and Node.js v25 is neither of those. I'm not opposed to adding it to the CI necessarily, but this may give the impression that we "support" a runtime that we don't claim to.

I think it's good to test on the latest version. Node 25 in particular came with a bunch of breaking changes. It seems like a good idea to understand how those might impact the library since Node 26 will still include those breaking changes, and probably new ones.

That statement in the README is already a bit inaccurate anyway since Node 18 is supported but it has been EOL for some time now 😄

I think testing against Node.js 25 is okay. 🙂

While we don't officially support it, testing against it helps with two things:

• when the next LTS comes around, we already have addressed any problems that came up during the non-LTS version's development, so we're quicker to support the next LTS
• people will use it anyway (😅) and if we address issues on a best-effort basis while it's not supported, we reduce the amount of issues we need to triage - that helps cut down on noise.

We also did that with Node.js 21 and 23 IIRC.

@pichlermarc I'm not sure what's going on with the Windows tests results not being reported in the GitHub UI. It looks like they ran successfully - https://github.com/open-telemetry/opentelemetry-js/actions/runs/18654677064/job/53180745887?pr=6019

@cjihrig I think the status check update got lost somewhere in GitHub's infra. I re-started the test - PR will be added to the merge queue once it's done running. :)

Thank you for your contribution @cjihrig! 🎉 We would like to hear from you about your experience contributing to OpenTelemetry by taking a few minutes to fill out this survey.