Skip to content

Commit

Permalink
chore: remove peer deps (#44)
Browse files Browse the repository at this point in the history
* chore: remove peer deps

We added ipfs-http-client as a peer dep to signal to the user
that they should add the modules they depend on as deps of their project.

Starting with npm7 all peer deps get installed automatically which
defeats the purpose of our use of peer deps, so let's remove them.
  • Loading branch information
achingbrain authored Apr 9, 2021
1 parent 01524ac commit 4612728
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 3 deletions.
4 changes: 4 additions & 0 deletions .travis.yml
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,10 @@ os:
- linux
- osx

before_install:
# modules with pre-built binaries may not have deployed versions for bleeding-edge node so this lets us fall back to building from source
- npm install @mapbox/node-pre-gyp -g

script: npx nyc -s npm run test:node -- --bail
after_success: npx nyc report --reporter=text-lcov > coverage.lcov && npx codecov

Expand Down
3 changes: 0 additions & 3 deletions package.json
Original file line number Diff line number Diff line change
Expand Up @@ -36,9 +36,6 @@
"p-queue": "^6.3.0",
"peer-id": "^0.14.0"
},
"peerDependencies": {
"ipfs-http-client": "*"
},
"browser": {
"go-ipfs": false
},
Expand Down

0 comments on commit 4612728

Please sign in to comment.