Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update Github Actions Workflow #545

Open
Phillip9587 opened this issue Oct 17, 2024 · 2 comments · May be fixed by #546
Open

Update Github Actions Workflow #545

Phillip9587 opened this issue Oct 17, 2024 · 2 comments · May be fixed by #546

Comments

@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. Deprecation of Artifact Actions v3:

    • The actions/upload-artifact@v3 and actions/download-artifact@v3 actions are being deprecated as of November 30, 2024 (GitHub Deprecation Notice).
    • These actions should be updated to their latest versions to ensure continued functionality in the CI workflow.
@Phillip9587 Phillip9587 linked a pull request Oct 17, 2024 that will close this issue
@Phillip9587
Copy link
Contributor Author

  1. 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.

@Phillip9587
Copy link
Contributor Author

Phillip9587 commented Oct 17, 2024

  1. Minimum token permissions for the GITHUB_TOKEN:

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 a pull request may close this issue.

1 participant