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.check bitswap cid #49

Conversation

jcace
Copy link
Contributor

@jcace jcace commented Oct 19, 2023

No description provided.

jcace and others added 6 commits October 16, 2023 17:45
* first try

* spike: spade-traverse separate file

* revert: bitswap.go

* spike: working test retrieve & decode

* feat: extract findLinks fn

* spike: extract out SpadeTraversal

* spike: cleanup

* docs: add comment

* spike: rename

* refactor: errors.Wrap

* fix: PR comments

* fix: typo

* spike: fix for root node with only one link

* feat: merge spike code into bitswap.go

* feat: spade traversal - return retrievalResult

* feat: wire up SpadeTraversal to bs worker

* chore: cleanup, comments

* docs: update comment

* fix: resolve bsw client timeout issue

* cleanup: remove unused retrieve_size

* cleanup: use BitswapClient instead of creating new

* fix: FindLinks return empty array

* feat: rework traverse_depth -> max_traverse_depth

---------

Co-authored-by: Jason Cihelka <[email protected]>
* Update build.yml

Thinking that the images should be called spade-retrieval bot

* Update build.yml

adding auto fix to linter to make our lives easier.

* Update build.yml

only look at new changes

* Update build.yml

removing new seemed to cause a lot more errors

* Update build.yml

looking only for new issues in the code

* Update build.yml

* Update build.yml

Removing on lint on current file chagnes
@jcace
Copy link
Contributor Author

jcace commented Oct 19, 2023

Should be in Spade RB repo - closing this one

@jcace jcace closed this Oct 19, 2023
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