-
Notifications
You must be signed in to change notification settings - Fork 59
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
Still no entry for Node 33.0.2 #195
Comments
This is still an issue for me, the latest node-abi still doesn't work with 33.0.2. Is the electron-node-abi-updater[bot] working? I don't see a commit with an update to the registry. |
@VerteDinde it doesn't seem like the fix you tried to make for #189 had any effect. |
@cseickel, I can't reproduce your issue, it seems to work fine for Electron v33.0.2.
|
My apologies @dsanders11, when I dug deeper I realized that I had an older version of node-abi. 3.60 was the version implicitly used as a dependency of electron forge 7.5.1. When I explicitly installed node-abi and got version 3.71.0, then I got the correct answer when looking up Electron v33.0.2. Explicitly installing node-abi does not solve my problem though. The real issue here is that electron-forge uses [email protected], which uses [email protected], which is the true cause of my problem. 3.7.1 is the latest version of rebuild, so I guess that means electron forge is just not compatible with the 33.x series of electron releases yet. All of this is not a fault of this project, so I will close the issue. Hopefully the above will help others troubleshoot in the future. |
@cseickel rebuild / forge does not depend on a specific version of node-abi, they depend on a range... You can update the transitive dependency yourself quite easily with either Forge and Rebuild won't publish new versions every time node-abi does, that would be absurd, instead apps should update their dependency tree accordingly. |
Thanks for the info @MarshallOfSound. I use yarn for the project with the issue and What worked for me was deleting |
Using yarn@1 you can use a tool I wrote for this. |
This is a repeat of #189. It was closed as fixed but no fix was actually made that I can see. There has also been no new releases since that issue was opened. If code was checked in that should fix it, maybe it just needs to be released?
The text was updated successfully, but these errors were encountered: