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

add entries for Node 12 and Electron 5 #62

Merged
merged 4 commits into from Apr 25, 2019
Merged

add entries for Node 12 and Electron 5 #62

merged 4 commits into from Apr 25, 2019

Conversation

shiftkey
Copy link
Contributor

@shiftkey shiftkey commented Apr 23, 2019

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.

@shiftkey shiftkey changed the title add entry for Node 12 add entries for Node 12 and Electron 5 Apr 24, 2019
index.js Show resolved Hide resolved
@jacobq
Copy link
Contributor

jacobq commented Apr 24, 2019

Would you mind also updating the test suite for getAbi w/ electron?

  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>
@jacobq
Copy link
Contributor

jacobq commented Apr 24, 2019

❤️

@lgeiger Any reason not to merge? Do we need to update the dates/logic regarding lts?

@malept
Copy link
Member

malept commented Apr 25, 2019

Hrm, this didn't trigger a new release. Possibly because the commit message wasn't in the proper format?

@lgeiger
Copy link
Contributor

lgeiger commented Apr 25, 2019

🎉 This PR is included in version 2.8.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@malept
Copy link
Member

malept commented Apr 25, 2019

Hm, I guess I was too impatient 🤷‍♂️

@MarshallOfSound
Copy link
Member

@malept I manually kicked it with an empty release 😆

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

Successfully merging this pull request may close these issues.

None yet

5 participants