-
Notifications
You must be signed in to change notification settings - Fork 19
add 'node-pre-gyp' installation to 'check' and 'test-node' actions #152
Conversation
I can't manage Windows CI to pass. Please advice if you can. |
It looks like windows installation problem with node-pre-gyp |
Yeah. This I understand but I have no clue why it happens. The package author recommends to switch to |
Ok, it's ready. Please squash the PR when merging. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
❤️ Thanks
@vasco-santos thank you. I see that CI failed on the merged PR.
|
@achingbrain can you please setup token? I think we also need NPM token? |
## [1.0.0](v0.7.1...v1.0.0) (2022-03-28) ### ⚠ BREAKING CHANGES * this module is now ESM-only Co-authored-by: achingbrain <[email protected]> ### Features * convert to typescript ([#151](#151)) ([85ce5cf](85ce5cf)) ### Bug Fixes * add 'node-pre-gyp' installation to 'check' and 'test-node' actions ([#152](#152)) ([bf4a68b](bf4a68b)) ### Trivial Changes * add note about `node-pre-gyp` to readme.md ([#141](#141)) ([ab4cc82](ab4cc82)), closes [#140](#140) * **deps-dev:** bump aegir from 35.2.1 to 36.0.0 ([#139](#139)) ([720cfad](720cfad)) * replace Travis with Github Actions ([#150](#150)) ([a73735b](a73735b)) * update project config ([13ab340](13ab340)) * update Readme ([#148](#148)) ([ba9facb](ba9facb))
🎉 This PR is included in version 1.0.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
cc @vasco-santos