ChainSafe / ChainSafe/fil-actor-states

[Upstream] deal_provider_collateral_bounds improvements.

Open
#280 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Rust
Stars
6
Forks
4
PR merge metrics
No merged PRs in 30d

Description

1. Take a more readable approach for the method.
2. Perhaps introduce that same method for actors v8..v10.

You're welcome to propose a change in the upstream https://github.com/filecoin-project/builtin-actors to address readability.

_Originally posted by @LesnyRumcajs in https://github.com/ChainSafe/fil-actor-states/pull/272#discussion_r1551491575_

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by locating the deal_provider_collateral_bounds method and reviewing the upstream builtin-actors discussion linked in the issue. Clarify a more readable approach there, then determine whether the same method should be introduced for actors v8 through v10; done means the intended readability improvement and version coverage are agreed and implemented.

Written by the indexing model from the issue text.

Assessment

Tech stack
rust
Domain
blockchain
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.