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(ipld): use bindnode/registry #340

Merged
merged 2 commits into from
Jun 25, 2022
Merged

feat(ipld): use bindnode/registry #340

merged 2 commits into from
Jun 25, 2022

Conversation

rvagg
Copy link
Member

@rvagg rvagg commented Jun 16, 2022

Using the v0.18 tracking branch @ ipld/go-ipld-prime#444

I had to play a bit of dependency wac-a-mole to get this working, libp2p stuff outdated, conflicts surrounding go-multicodec iirc.

@codecov-commenter
Copy link

Codecov Report

Merging #340 (9fd772e) into v2 (c833220) will increase coverage by 0.40%.
The diff coverage is 56.25%.

Impacted file tree graph

@@            Coverage Diff             @@
##               v2     #340      +/-   ##
==========================================
+ Coverage   71.18%   71.58%   +0.40%     
==========================================
  Files          21       21              
  Lines        2717     2699      -18     
==========================================
- Hits         1934     1932       -2     
+ Misses        576      562      -14     
+ Partials      207      205       -2     
Impacted Files Coverage Δ
message/message1_1prime/message.go 50.00% <50.00%> (ø)
message/message1_1prime/transfer_request.go 63.49% <50.00%> (+4.07%) ⬆️
message/message1_1prime/transfer_response.go 86.66% <50.00%> (+8.23%) ⬆️
message/message1_1prime/transfer_message.go 33.33% <66.66%> (+11.11%) ⬆️

Copy link
Collaborator

@hannahhoward hannahhoward left a comment

Choose a reason for hiding this comment

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

LGTM

@hannahhoward hannahhoward merged commit 61f0756 into v2 Jun 25, 2022
@rvagg rvagg deleted the rvagg/bindnode-registry branch June 25, 2022 12:04
hannahhoward added a commit that referenced this pull request Oct 7, 2022
* chore(deps): update libp2p v0.19.4 (#341)

* feat(ipld): use bindnode/registry

Co-authored-by: Hannah Howard <hannah@hannahhoward.net>
hannahhoward added a commit that referenced this pull request Oct 8, 2022
* chore(v2): setup the v2 release

given the expected breaking changes, it's time to setup the v2 release

BREAKING CHANGE: v2 modules

* Refactor revalidators (#308)

* feat(revalidators): initial refactor

refactor revalidation process -- removing independent revalidations, making validations results more
clear

* refactor(rename): RevalidateToComplete -> RequiresFinalization

* refactor(datatransfer): enhance validator comments

* refactor(message): revert message response changes

* refactor(impl): comment and refactor on events

add comments to event processing and also extract functions for receiving requests to a new file

* refactor(message): s/IsVoucherResult/IsValidationResult

rename the IsVoucherResult to is 'IsValidationResult' also add a method for generating messages from
validation results

* feat(events): only dispatch events on change

Only dispatch Pause, Resume, SetDataLimit, and RequiresFinalization on change

* style(imports): fix imports

* feat(events): add DataLimitExceeded event

* Update channels/channel_state.go

Co-authored-by: dirkmc <dirkmdev@gmail.com>

* Update manager.go

Co-authored-by: dirkmc <dirkmdev@gmail.com>

* Update manager.go

Co-authored-by: dirkmc <dirkmdev@gmail.com>

* Update statuses.go

Co-authored-by: Rod Vagg <rod@vagg.org>

Co-authored-by: dirkmc <dirkmdev@gmail.com>
Co-authored-by: Rod Vagg <rod@vagg.org>

* Refactor revalidators v2 (#322)

* refactor(validators): remove revalidation

move to all revalidation being asynchronous

* feat(validators): implied pauses

causes datalimit exceeded and requires finalization to leave request paused, regardless of where
LeaveRequestPaused is set

* refactor(events): reorder events

reorder validation events so all get record when transfer finishes

* Update impl/impl.go

Co-authored-by: Rod Vagg <rod@vagg.org>

Co-authored-by: Rod Vagg <rod@vagg.org>

* chore(message): delete old message format code (#330)

* feat(ipld): vouchers as plain ipld.Node (#325)

* feat(ipld): vouchers as plain ipld.Node

* feat: add ValidationResult#Equals() utility

* feat(ipld): introduce TypedVoucher tuple type

* chore(ipld): ipld.Node -> datamodel.Node

* chore: remove RegisterVoucherResultType

* fix: minor staticcheck fixes

* refactor(channelstate): use cborgencompatiblenode (#336)

use cborgencomptaiblenode to simply channelstate interface

* feat(ipld): use bindnode/registry (#340)

* chore(deps): update libp2p v0.19.4 (#341)

* feat(ipld): use bindnode/registry

Co-authored-by: Hannah Howard <hannah@hannahhoward.net>

* refactor(v2): port graphsync w/o transport refactor

Ports state machine and graphsync changes without the big transport refactor

* fix(pr): respond to pr comments

* chore(deps): upgrade libp2p v0.22

also upgrades graphsync and removes go-libp2p-core paths

* fix(deps): use tagged go-ipld-prime

Co-authored-by: dirkmc <dirkmdev@gmail.com>
Co-authored-by: Rod Vagg <rod@vagg.org>
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.

3 participants