Skip to content

ci: updated github actions ci workflow#69

Merged
wesleytodd merged 1 commit intojshttp:masterfrom
Phillip9587:ci
Mar 5, 2025
Merged

ci: updated github actions ci workflow#69
wesleytodd merged 1 commit intojshttp:masterfrom
Phillip9587:ci

Conversation

@Phillip9587
Copy link
Contributor

I noticed that the current CI workflow in this repository could benefit from some updates. Specifically:

  1. Node.js Installation in the Matrix Test Run:

    • Currently, the workflow uses nvm to install Node.js versions.
    • It might be worth considering switching to the official actions/setup-node action. This action can leverage cached Node.js versions from the runner, which could improve efficiency and speed up the CI pipeline.
  2. The Coverage setup could also be optimized:

    • Currently, the workflow uses the coverallsapp/github-action@master which points to v1 of this action. This v1 action uses node16 as runtime which is deprecated.
  3. Minimum token permissions for the GITHUB_TOKEN:

  4. Separate Lint Step

@wesleytodd wesleytodd merged commit 3a77a9c into jshttp:master Mar 5, 2025
10 checks passed
@UlisesGascon UlisesGascon mentioned this pull request Nov 18, 2025
@Phillip9587 Phillip9587 deleted the ci branch February 10, 2026 18:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants