-
Notifications
You must be signed in to change notification settings - Fork 60
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
add entries for Node 12 and Electron 5 #62
Conversation
Would you mind also updating the test suite for t.equal(getAbi('5.0.0', 'electron'), '70')
t.equal(getAbi('4.1.4', 'electron'), '69')
t.equal(getAbi('4.0.4', 'electron'), '69')
t.equal(getAbi('4.0.3', 'electron'), '64')
t.equal(getAbi('3.1.8', 'electron'), '64')
t.equal(getAbi('2.0.18', 'electron'), '57') |
Co-Authored-By: Jacob <jacobq@users.noreply.github.com>
❤️ @lgeiger Any reason not to merge? Do we need to update the dates/logic regarding |
Hrm, this didn't trigger a new release. Possibly because the commit message wasn't in the proper format? |
🎉 This PR is included in version 2.8.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
Hm, I guess I was too impatient 🤷♂️ |
@malept I manually kicked it with an empty release 😆 |
I think the details are correct here, based on the details in https://nodejs.org/en/download/releases/. and https://github.com/nodejs/node/blob/master/doc/abi_version_registry.json
Let me know if anything is unclear.