-
Notifications
You must be signed in to change notification settings - Fork 50
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 canvas, regenerate lock #17
base: master
Are you sure you want to change the base?
Conversation
Is there a reason this one isn't being merged and releasing a new version of this package? This allows this package to be built at all on node >= v13 due to this issue which is solved in canvas v2.6.1: Automattic/node-canvas#1485 cc @jo |
package.json
Outdated
@@ -16,7 +16,7 @@ | |||
"url": "https://github.com/null2/color-thief/issues" | |||
}, | |||
"dependencies": { | |||
"canvas": "~2.5.0" | |||
"canvas": "~2.6.0" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"canvas": "~2.6.0" | |
"canvas": "~2.6.1" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You will have to update the package-lock.json again.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Mike-Dunton Already resolves to the 2.6.1 vers. https://github.com/null2/color-thief/pull/17/files#diff-32607347f8126e6534ebc7ebaec4853dR356
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Whoops. I was wrong. ~ include everything greater than a particular version in the same minor range.
[Enhancement] Canvas bump
No description provided.