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

Update ipld-dag-pb to the latest version 🚀 #2307

Closed
wants to merge 1 commit into from

Conversation

greenkeeper[bot]
Copy link
Contributor

@greenkeeper greenkeeper bot commented Jul 29, 2019

The dependency ipld-dag-pb was updated from 0.17.4 to 0.18.0.

This version is not covered by your current version range.

If you don’t accept this pull request, your project will work just like it did before. However, you might be missing out on a bunch of new features, fixes and/or performance improvements from the dependency update.


Publisher: vmx
License: MIT

Release Notes for v0.18.0

Bug Fixes

Features

Performance Improvements

  • remove manual enumerability modifications (37ffdd5), closes #152
  • remove named links from object (4dbe00d)

BREAKING CHANGES

  • addLink() and rmLink() are now instance methods.

Prior to this change:

DAGNode.addLink(node, link)
DAGNode.rmLink(node, name)

Now:

node.addLink(link)
node.rmLink(name)
  • It's no longer possible to pass a DAGNode into addLink().

Intead of passing in a DAGNode into addLink(), convert that node into
a DAGLink via toDAGLink().

Example:

Prior to this change:

const node = new DAGNode('some data')
const node2 = new DAGNode('use that as link')
await DAGNode.addLink(node, node2)

Now:

const node = new DAGNode('some data')
const node2 = new DAGNode('use that as link')
DAGNode.addLink(node, await node2.toDAGLink())
  • DAGNode.create() is removed

Instead of DAGNode.create(), please use new DAGNode() instead. It
takes the same parameters and is compatible to create().

Example:

Prior to this change:

const node = DAGNode.create('some data', links)

Now:

const node = new DAGNode('some data', links)
  • DAGNode.clone() is removed from public API without any replacement.

Also the API for rmLink() and addLink() changed. They no longer
return a new node, but just remove/add the links to/from the current
node.

Prior to this change:

const lessLinks = DAGNode.rmLink(node1, 'Link1')
node1 = lessLinks
const moreLinks = await DAGNode.addLink(node2, link)
node2 = moreLinks

Now:

DAGNode.rmLink(node, 'Link1')
await DAGNode.addLink(node2, link)
  • named links are no longer part of an object

Access to named links is only possible with calling resolve().
Hence they are also not part of tree() anymore.

Named links are a feature of IPFS and only supported for
backwards compatibility, they are not really part of IPLD.

Commits

The new version differs by 14 commits.

  • 18be385 chore: release version v0.18.0
  • bcbd9c0 chore: update contributors
  • 3b6e569 chore: update dependencies
  • 37ffdd5 perf: remove manual enumerability modifications
  • a9aa0a0 feat: make addLink()/rmLink() instance methods
  • 7f1a00a fix: make addLink() synchronous
  • 029174d feat: remove DAGNode.create()
  • d5e1135 feat: remove cloning
  • 8630de5 chore(package): update aegir to version 20.0.0
  • 4dbe00d perf: remove named links from object
  • 5d43cec chore: update dependencies
  • 4dc77c0 chore: remove .js extensions, explicit specifiers
  • 9201772 chore: refactor internal circular dependencies
  • 8d8a4cf chore: remove dependency on async

See the full diff


FAQ and help

There is a collection of frequently asked questions. If those don’t help, you can always ask the humans behind Greenkeeper.


Your Greenkeeper bot 🌴

@alanshaw
Copy link
Member

@vmx any chance you can help us migrate to this new version?

@vmx
Copy link
Member

vmx commented Jul 30, 2019

Sure. The work for UnixFS is already done, locally (I'm just waiting for a new js-ipld release to base those PRs on). I haven't looked into js-ipfs, yet. I'd do those once the UnixFS things are approved.

@vmx
Copy link
Member

vmx commented Aug 5, 2019

So the changes for UnixFS were straight forward. The problem is js-ipfs. You also would want to upgrade ipld. But this means you also need to upgrade:

  • ipfs-repo
  • interface-datastore
  • ipfs-blockservice

Those upgrades also need an upgrade of those dependencies in libp2p.

@achingbrain
Copy link
Member

The unixfs-* and mfs PRs have all been merged & released.

greenkeeper bot added a commit that referenced this pull request Aug 19, 2019
@greenkeeper
Copy link
Contributor Author

greenkeeper bot commented Aug 19, 2019

  • The dependency ipld-dag-pb was updated from 0.17.4 to 0.18.1.

Update to this version instead 🚀

Release Notes for v0.18.1

Bug Fixes

  • serialization and size portability problems (f348cb8)
Commits

The new version differs by 3 commits.

  • 1cb217d chore: release version v0.18.1
  • 75a28c3 chore: update contributors
  • f348cb8 fix: serialization and size portability problems

See the full diff

@achingbrain achingbrain deleted the greenkeeper/ipld-dag-pb-0.18.0 branch April 7, 2020 14:42
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants