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

deps: update node-canvas, Node, actions, + npm audit fix #115

Merged
merged 1 commit into from
Dec 31, 2024

Conversation

agilgur5
Copy link
Owner

@agilgur5 agilgur5 commented Dec 31, 2024

Summary

Update node-canvas and CI matrix to use non-EoL Node

Details

  • update node-canvas to latest v3 to work with newer, non-EoL Node versions
    • also less deps like node-pre-gyp etc
  • update CI matrix to use LTS Node 18+
  • update actions/checkout and actions/setup-node to use non-EoL Node runtimes as well
  • run npm audit fix after

Notes to Reviewers

  • also need to update branch protection to use Node 18 as a check (instead of Node 16)

- update `node-canvas` to latest v3 to work with newer, non-EoL Node versions
  - also less deps like `node-pre-gyp` etc
- update CI matrix to use LTS Node 18+
- run `npm audit fix` after
@agilgur5 agilgur5 added kind: internal Changes only affect the internals and not the API or usage scope: dependencies Pull requests that update a dependency file labels Dec 31, 2024

This comment was marked as resolved.

@agilgur5 agilgur5 changed the title deps: update node-canvas, Node, + npm audit fix deps: update node-canvas, Node, actions, + npm audit fix Dec 31, 2024
Copy link
Owner Author

@agilgur5 agilgur5 left a comment

Choose a reason for hiding this comment

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

✅ CI all green with the new matrix and deps and changes LGTM

@agilgur5 agilgur5 merged commit cf5b7c8 into main Dec 31, 2024
8 checks passed
@agilgur5 agilgur5 deleted the deps-update-node-canvas-etc branch December 31, 2024 21:31
agilgur5 added a commit that referenced this pull request Jan 3, 2025
deps: update `node-canvas`, Node, + `npm audit fix`

- update `node-canvas` to latest v3 to work with newer, non-EoL Node versions
  - also less deps like `node-pre-gyp` etc
- update CI matrix to use LTS Node 18+
- update `actions/checkout` and `actions/setup-node` to use non-EoL Node runtimes as well
  - the primary breaking change in updating [both to their](https://github.com/actions/checkout/releases/tag/v4.0.0) [respective v4 releases](https://github.com/actions/setup-node/releases/tag/v4.0.0) is the Node runtime update
- run `npm audit fix` after

(cherry picked from commit cf5b7c8)
- fixed merge conflicts with other surrounding deps
  - basically only `node-canvas` updated and nothing else
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind: internal Changes only affect the internals and not the API or usage scope: dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant