Skip to content
This repository has been archived by the owner on Feb 12, 2024. It is now read-only.

fix: update to new aegir #3528

Merged
merged 60 commits into from
Mar 7, 2021
Merged

fix: update to new aegir #3528

merged 60 commits into from
Mar 7, 2021

Conversation

hugomrdias
Copy link
Member

@hugomrdias hugomrdias commented Feb 4, 2021

  • update aegir
  • fix some linting
  • enable multiple test runs at the same time (all servers now try to find free ports)

todo:

  • ipfs-message-port-client

needs:
ipfs/interop#223

- name: Bundlesize ${{ matrix.project }}
uses: ipfs/aegir/actions/bundle-size@v29.0.0
uses: ipfs/aegir/actions/bundle-size@master
Copy link
Member

Choose a reason for hiding this comment

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

This should go back to being a version number

const notAvailablePeerId = 'QmUmaEnH1uMmvckMZbh3yShaasvELPW4ZLPWnB4entMTEn'
const count = 2

return expect(all(ipfsA.ping(notAvailablePeerId, { count }))).to.eventually.be.rejected()
Copy link
Member

Choose a reason for hiding this comment

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

Why was this necessary?

Copy link
Member Author

Choose a reason for hiding this comment

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

i was trying to catch an uncaught rejection that happens sometimes, because the new runner fails when these happen now.
this seems to help a bit but 100%

@@ -45,15 +45,14 @@
"test:chrome": "aegir test -t browser -t webworker -- --browsers ChromeHeadless",
"test:firefox": "aegir test -t browser -t webworker -- --browsers FirefoxHeadless",
"lint": "aegir lint",
"prepare": "aegir build --no-bundle",
Copy link
Member

Choose a reason for hiding this comment

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

Why was this removed?

Copy link
Member Author

Choose a reason for hiding this comment

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

we dont need it here, the prepare for types is just to help with PR with gh url and mono-repo doesnt allow for gh url dep install. Plus it slows down everything

Copy link
Member

Choose a reason for hiding this comment

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

mono-repo doesnt allow for gh url dep install

Good news! Now you can do:

$ npm install https://gitpkg.now.sh/ipfs/js-ipfs/packages/ipfs-core?some-branch-identifier

https://gitpkg.now.sh

"coverage": "npx nyc -r html npm run test:node -- --bail",
"clean": "rimraf ./dist",
"dep-check": "aegir dep-check -i ipfs-core -i rimraf -i ipfs-core-types"
"dep-check": "aegir dep-check -i ipfs-core -i rimraf -i ipfs-core-types -i esbuild"
Copy link
Member

Choose a reason for hiding this comment

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

esbuild isn't a dep here?

Copy link
Member Author

Choose a reason for hiding this comment

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

dep-check doesnt look into .aegir.js

@achingbrain
Copy link
Member

The fs module prints out:

I'm `fs` modules

in the browser console, needs to be overridden to false or something.

@hugomrdias
Copy link
Member Author

The fs module prints out:

I'm `fs` modules

in the browser console, needs to be overridden to false or something.

https://github.com/ipfs-shipyard/js-mock-ipfs-pinning-service > https://github.com/isa-group/oas-tools/blob/master/package.json#L26 > https://github.com/npm/security-holder > open node_modules/fs/index.js 🤷🏻‍♂️

@achingbrain achingbrain merged commit 49f7880 into master Mar 7, 2021
@achingbrain achingbrain deleted the feat/update-to-new-aegir branch March 7, 2021 13:17
@achingbrain achingbrain added this to v0.55.0 Better Types in ipfs/js-ipfs Project Roadmap Apr 1, 2021
SgtPooki referenced this pull request in ipfs/js-kubo-rpc-client Aug 18, 2022
- update aegir
- fix some linting
- enable multiple test runs at the same time (all servers now try to find free ports)

Co-authored-by: achingbrain <alex@achingbrain.net>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
ipfs/js-ipfs Project Roadmap
v0.55.0 Better Types
Archived in project
Development

Successfully merging this pull request may close these issues.

None yet

2 participants