Skip to content
This repository has been archived by the owner on Sep 30, 2023. It is now read-only.

preparations for js-ipfs 0.33 #183

Closed
alanshaw opened this issue Nov 1, 2018 · 1 comment · Fixed by #185
Closed

preparations for js-ipfs 0.33 #183

alanshaw opened this issue Nov 1, 2018 · 1 comment · Fixed by #185
Assignees

Comments

@alanshaw
Copy link

alanshaw commented Nov 1, 2018

👋 I upgraded ipfs to 0.33.0-rc.4, datastore-level to ^0.9.0, ipfs-repo to ^0.24.0 and ran the tests and they all passed \o/

ipfs/js-ipfs#1635

@aphelionz
Copy link
Contributor

aphelionz commented Nov 3, 2018

First ran the tests with master:

$ git fetch && git pull origin master
From https://github.com/orbitdb/ipfs-log
 * branch            master     -> FETCH_HEAD
Already up to date.
$ rm -rf node_modules && npm install && npm run test
290 passing (2m) # All tests pass!

Then I applied @alanshaw's upgrades...

diff --git a/package.json b/package.json
index ca68ee2..935d980 100644
--- a/package.json
+++ b/package.json
@@ -24,7 +24,7 @@
     "babel-loader": "^8.0.2",
     "babel-plugin-transform-runtime": "^6.23.0",
     "babel-preset-env": "^1.6.1",
-    "datastore-level": "~0.8.0",
+    "datastore-level": "^0.9.0",
     "eslint": "^5.4.0",
     "eslint-config-standard": "^12.0.0",
     "eslint-plugin-import": "^2.14.0",
@@ -32,8 +32,8 @@
     "eslint-plugin-promise": "^4.0.0",
     "eslint-plugin-standard": "^4.0.0",
     "go-ipfs-dep": "~0.4.17",
-    "ipfs": "~0.31.4",
-    "ipfs-repo": "~0.23.1",
+    "ipfs": "~0.33.0",
+    "ipfs-repo": "^0.24.0",
     "ipfsd-ctl": "~0.39.1",
     "istanbul": "^0.4.5",
     "mocha": "^5.2.0",

...and ran npm install again:

$ rm -rf node_modules && npm install && npm run test
290 passing (1m) # All tests pass, and perhaps a bit faster?

Looks good! Confirmed that this is all working. Addressed in #185

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

Successfully merging a pull request may close this issue.

3 participants