atharvalade opened a new pull request, #2792: URL: https://github.com/apache/iggy/pull/2792
## Which issue does this PR close? Closes #2774 ## Rationale Rust and Java already upload coverage to Codecov — Node SDK was missing. ## What changed? Replaced `npm run test:unit` / `npm run test:e2e` in the pre-merge action with direct `node` invocations that add `--test-reporter=lcov --test-reporter-destination=../../reports/node-coverage/lcov.info`, using Node 23's native lcov support (no new dependencies). Added Codecov upload steps in `_test.yml` (pre-merge) and a `node-coverage` baseline job in `post-merge.yml`, following the same pattern as Java. Registered the `node` flag in `codecov.yml` and removed `foreign/node/**` from the ignore list. ## Local Execution - CI-only workflow changes, no local server setup - Pre-commit hooks ran ## AI Usage 1. Opus 4.6 2. Used to scaffold the workflow YAML following existing Java/Rust patterns 3. Verified by diffing against existing Java coverage steps and reviewing all four files 4. Yes -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
