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

Upgrade canvas dependency to v2.6.1 - This fixes incompatibility with recent versions of node #19

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

pineapplemachine
Copy link

@pineapplemachine pineapplemachine commented Nov 18, 2020

What kind of change does this PR introduce?

This is a bugfix.

Does this PR introduce a breaking change?

No.

Did you test your solution?

I ran npm test. An MSDF image was successfully produced and written to the test folder.

Problem Description

The version of canvas used in the current version of msdf-bmfont is incompatible with recent versions of node.

I encountered a similar error as was reported here, so I tried upgrading this package's version of canvas. It worked as hoped and now I am able to use msdf-bmfont.

Automattic/node-canvas#1501 (comment)

Solution Description

The version of the canvas package dependency has been updated to a version that is compatible with recent versions of node.

Side Effects, Risks, Impact

Node 6.x and lower will no longer be supported by this tool.

https://github.com/Automattic/node-canvas/blob/master/CHANGELOG.md#200

Aditional comments:

None.

Prior version no longer works with newer versions of node
Package version upgrade introduced a breaking change
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.

1 participant