Skip to content

Conversation

@Phillip9587
Copy link
Contributor

This PR updates the CI pipeline to match the main express one:

  • separate lint step
  • test matrix with all supported node versions
  • removed legacy coverage and test depdency checks
  • use actions/setup-node to install Node.js
  • use coverallsapp/github-action v2

This PR also removes --bail from the test script and updates the test dependencies and applies the caret notation to them.

@Phillip9587 Phillip9587 requested a review from UlisesGascon July 23, 2025 16:35
Copy link
Member

@UlisesGascon UlisesGascon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@UlisesGascon UlisesGascon self-assigned this Jul 24, 2025
@UlisesGascon UlisesGascon merged commit 1fe49ae into jshttp:master Jul 24, 2025
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants