Skip to content
This repository was archived by the owner on Apr 24, 2023. It is now read-only.

add 'node-pre-gyp' installation to 'check' and 'test-node' actions #152

Merged
merged 9 commits into from
Mar 15, 2022

Conversation

vogdb
Copy link
Contributor

@vogdb vogdb commented Mar 14, 2022

@vogdb vogdb changed the title move 'node-pre-gyp' installation after 'setup-node@v2' move 'node-pre-gyp' installation to 'check' action Mar 14, 2022
@vogdb
Copy link
Contributor Author

vogdb commented Mar 14, 2022

I can't manage Windows CI to pass. Please advice if you can.

@vasco-santos
Copy link
Member

It looks like windows installation problem with node-pre-gyp

@vogdb
Copy link
Contributor Author

vogdb commented Mar 14, 2022

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 @mapbox/node-pre-gyp. I will try it.

@vogdb vogdb changed the title move 'node-pre-gyp' installation to 'check' action add 'node-pre-gyp' installation to 'check' and 'test-node' actions Mar 15, 2022
@vogdb
Copy link
Contributor Author

vogdb commented Mar 15, 2022

Ok, it's ready. Please squash the PR when merging.

Copy link
Member

@vasco-santos vasco-santos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

❤️ Thanks

@vasco-santos vasco-santos merged commit bf4a68b into libp2p:master Mar 15, 2022
@vogdb vogdb deleted the fix-github-actions branch March 15, 2022 08:41
@vogdb
Copy link
Contributor Author

vogdb commented Mar 15, 2022

@vasco-santos thank you. I see that CI failed on the merged PR. AEGIR_GHTOKEN needs to be set. Please look.

8s
Run npm run --if-present release

> [email protected] release
> aegir release --target node --target browser

Missing GitHub access token. Have you set `AEGIR_GHTOKEN`?
Error: Process completed with exit code 1.

@vasco-santos
Copy link
Member

@achingbrain can you please setup token? I think we also need NPM token?

github-actions bot pushed a commit that referenced this pull request Mar 28, 2022
## [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))
@github-actions
Copy link

🎉 This PR is included in version 1.0.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

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

Successfully merging this pull request may close these issues.

3 participants