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

Added support Node 12.x.x & 13.x.x. Updated deprecated functions for v8. #523

Merged
merged 13 commits into from
Nov 24, 2019

Conversation

oktapodia
Copy link
Collaborator

No description provided.

@dsteinman
Copy link

dsteinman commented Nov 23, 2019

I can confirm this branch works on MacOS with Node v12.8.1 and Electron 7.1.2, thanks very much!

@oktapodia oktapodia changed the title Fix/node13 Added support Node 12.x.x & 13.x.x. Updated deprecated functions for v8. Nov 24, 2019
@oktapodia oktapodia marked this pull request as ready for review November 24, 2019 13:48
@oktapodia oktapodia merged commit 7dc5ab4 into master Nov 24, 2019
@oktapodia oktapodia deleted the fix/node13 branch November 24, 2019 13:51
@RobbieTheWagner
Copy link

Could we get a new release that includes this please?

@oktapodia
Copy link
Collaborator Author

Will do as soon as possible

@oktapodia oktapodia added this to the v0.6.0 milestone Nov 24, 2019
@dsteinman
Copy link

Could we get a new release that includes this please?

Until the release is made you can change your package.json to point to the master branch:

"dependencies": {
    "robotjs": "git+https://github.com/octalmage/robotjs.git#master"

@RobbieTheWagner
Copy link

@dsteinman yeah, I have done that. I just prefer to use a released version.

comcaptain added a commit to comcaptain/video-tagger that referenced this pull request Dec 8, 2019
I used master because the change is not released yet, check octalmage/robotjs#523 for detail
@jorgecuevas92
Copy link

Works on Ubuntu 18.04 and Node v12.13.1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants