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

Updates #103

Merged
merged 2 commits into from
Dec 7, 2022
Merged

Updates #103

merged 2 commits into from
Dec 7, 2022

Conversation

adam-maj
Copy link
Contributor

@adam-maj adam-maj commented Dec 7, 2022

No description provided.

@adam-maj adam-maj merged commit e18eb09 into main Dec 7, 2022
@adam-maj adam-maj deleted the am/test branch December 7, 2022 19:12
adam-maj added a commit that referenced this pull request Dec 9, 2022
* Updates (#103)

* pass context in a few more places (#101)

* handle and return err (#102)

Co-authored-by: ricebin <ricebin@users.noreply.github.com>

* ipfs related cleanups
1. reuse httpclient when possible
2. pass context in more places
3. dont forget to check errors

Co-authored-by: Adam Majmudar <64697628+adam-maj@users.noreply.github.com>
adam-maj added a commit that referenced this pull request Dec 9, 2022
* Updates (#103)

* pass context in a few more places (#101)

* handle and return err (#102)

Co-authored-by: ricebin <ricebin@users.noreply.github.com>

* ipfs related cleanups
1. reuse httpclient when possible
2. pass context in more places
3. dont forget to check errors

Co-authored-by: Adam Majmudar <64697628+adam-maj@users.noreply.github.com>

Co-authored-by: ricebin <ricebin@users.noreply.github.com>
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 this pull request may close these issues.

2 participants