Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: client.get uses unpackStream with blockstore #13

Closed
wants to merge 7 commits into from

Conversation

olizilla
Copy link
Contributor

@olizilla olizilla commented Jul 1, 2021

  • Update client.get to use feat: add unpackStream storacha/ipfs-car#46
    • uses a blockstore during unpack so should be able to deal with cars larger than mem now.
    • ipfs-car adopts the ReadableStream vs AsyncIterator shim which is nice
  • Adds clean target called before build to ensure we're fresh.

Needs web3-storage/web3-file#7

Builds on #12

License: (Apache-2.0 AND MIT)
Signed-off-by: Oli Evans oli@tableflip.io

Base automatically changed from feat/add-car to main July 2, 2021 08:08
vasco-santos and others added 3 commits July 2, 2021 09:19
- Update client.get to use unpackStream from ipfs-car
  - uses a blockstore during unpack so should be able to deal with cars larger than mem now.
  - ipfs-car adopts the ReadableStream vs AsyncIterator shim which is nice
  - storacha/ipfs-car#46

License: (Apache-2.0 AND MIT)
Signed-off-by: Oli Evans <oli@tableflip.io>
License: (Apache-2.0 AND MIT)
Signed-off-by: Oli Evans <oli@tableflip.io>
License: (Apache-2.0 AND MIT)
Signed-off-by: Oli Evans <oli@tableflip.io>
License: (Apache-2.0 AND MIT)
Signed-off-by: Oli Evans <oli@tableflip.io>
License: (Apache-2.0 AND MIT)
Signed-off-by: Oli Evans <oli@tableflip.io>
@olizilla
Copy link
Contributor Author

olizilla commented Jul 2, 2021

Closing in favour of the mergable #16

@olizilla olizilla closed this Jul 2, 2021
orvn pushed a commit that referenced this pull request Dec 1, 2021
feat Logins page, buttons refactor, new variants, variable updates
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants