update github actions #53
Conversation
There was a problem hiding this comment.
Pull request overview
This PR updates the GitHub Actions workflow configurations to standardize Node.js version and ensure the latest npm version is used across all workflows, along with a package version bump.
- Standardizes Node.js version to 24 across all workflow jobs
- Adds explicit npm upgrade step to ensure latest npm version is used
- Bumps package version from 0.0.23 to 0.0.24
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| package.json | Version bump from 0.0.23 to 0.0.24 |
| .github/workflows/publish.yml | Adds npm upgrade step before installation |
| .github/workflows/lint.yml | Standardizes Node.js version to 24 and adds npm upgrade step for both macOS and Windows test jobs |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| - uses: actions/setup-node@v6 | ||
| with: | ||
| node-version: 24 | ||
| registry-url: "https://registry.npmjs.org" |
There was a problem hiding this comment.
The registry-url configuration is unnecessary for the lint workflow since it's only used when publishing to npm. This configuration is needed in publish.yml but not in lint.yml. Consider removing this line to keep the workflow configuration minimal and clear.
| - uses: actions/setup-node@v6 | ||
| with: | ||
| node-version: 24 | ||
| registry-url: "https://registry.npmjs.org" |
There was a problem hiding this comment.
The registry-url configuration is unnecessary for the lint workflow since it's only used when publishing to npm. This configuration is needed in publish.yml but not in lint.yml. Consider removing this line to keep the workflow configuration minimal and clear.
No description provided.