search

filecoin-project / go-state-types

Primitive and low level types used in chain state and actor method parameters
Other
24 stars 37 forks source link

Update deal state schema to match Rust #215

Closed anorth closed 1 year ago

anorth commented 1 year ago

210 doesn't yet include updating the market DealState schema. This PR isn't a complete resolution (I'm no longer familiar with the rest of the migration code), but intended to be cherry-picked or otherwise cloned into #210 as a prompt to migrate those objects.

See https://github.com/filecoin-project/builtin-actors/pull/1347 and https://github.com/filecoin-project/builtin-actors/pull/1403

anorth commented 1 year ago

Note that I have not ported changes to the state checks from Rust to Go. My understanding is that these have drifted in many places because no-one is maintaining the Go version. This is a predictable result of the code duplication involved. However, it would be good to prevent the problem getting worse.