Skip to content

Conversation

@achingbrain
Copy link
Member

When an imported file is big enough to need leaf nodes, and that
file is imported with metadata, we should only add that metadata
to the root of the file DAG and not to every leaf in the graph.

When an imported file is big enough to need leaf nodes, and that
file is imported with metadata, we should only add that metadata
to the root of the file DAG and not to every leaf in the graph.
@achingbrain achingbrain requested a review from Gozala February 1, 2022 07:02
@Gozala
Copy link
Contributor

Gozala commented Feb 3, 2022

@achingbrain I think this should probably go as feat! as it is going to be a breaking change causing different hashes to come out.

Copy link
Contributor

@Gozala Gozala left a comment

Choose a reason for hiding this comment

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

Thanks

@achingbrain achingbrain merged commit f5d3a67 into master Feb 4, 2022
@achingbrain achingbrain deleted the fix/do-not-add-metadata-to-leaves branch February 4, 2022 19:09
github-actions bot pushed a commit that referenced this pull request Apr 26, 2022
## [ipfs-unixfs-importer-v9.0.7](ipfs-unixfs-importer-v9.0.6...ipfs-unixfs-importer-v9.0.7) (2022-04-26)

### Bug Fixes

* do not add metadata to leaves ([#193](#193)) ([f5d3a67](f5d3a67))
* don't split paths on `^` characters ([#192](#192)) ([9a69908](9a69908))

### Trivial Changes

* switch to auto-release ([#208](#208)) ([99386e6](99386e6))
* switch to gh actions ([#175](#175)) ([395852b](395852b))
* update readmes ([#188](#188)) ([273a141](273a141))
@github-actions
Copy link

🎉 This PR is included in version ipfs-unixfs-importer-v9.0.7 🎉

The release is available on:

Your semantic-release bot 📦🚀

github-actions bot pushed a commit that referenced this pull request Apr 26, 2022
## [ipfs-unixfs-exporter-v7.0.7](ipfs-unixfs-exporter-v7.0.6...ipfs-unixfs-exporter-v7.0.7) (2022-04-26)

### Bug Fixes

* do not add metadata to leaves ([#193](#193)) ([f5d3a67](f5d3a67))

### Trivial Changes

* switch to auto-release ([#208](#208)) ([99386e6](99386e6))
* update readmes ([#188](#188)) ([273a141](273a141))
@github-actions
Copy link

🎉 This PR is included in version ipfs-unixfs-exporter-v7.0.7 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants