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

Node.js 12 support #133

Closed
wants to merge 3 commits into from
Closed

Node.js 12 support #133

wants to merge 3 commits into from

Conversation

fwcd
Copy link

@fwcd fwcd commented May 31, 2019

Migrate V8 API calls and update vulnerable dependencies using npm audit.

For reference: Some of the API changes are described here.

fwcd added 3 commits June 1, 2019 00:22
Migrate deprecated V8 APIs and update nan dependency.
Add an editors section to the .gitignore file and list the '.vscode'
folder.
@fwcd
Copy link
Author

fwcd commented May 31, 2019

The PR #120 seems to address the deprecation warnings.

@mboxed
Copy link

mboxed commented Aug 7, 2019

This PR doesn't seem to have been addressed for a couple of months. Would it be okay to friendly-fork this project with these changes?

Copy link

@ShenHongFei ShenHongFei left a comment

Choose a reason for hiding this comment

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

build successfully on windows 10 1803 with node v12.5.0

@cspotcode
Copy link

cspotcode commented Sep 21, 2019

Any chance we can get a canary / beta / rc build of this published to npm?

EDIT: I published a build myself.

https://npm.im/@cspotcode/speaker

I just pulled the fwcd:master, changed the package name, and ran npm publish. I hope it's useful until the official package is updated.

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.

None yet

4 participants