ChainSafe / ChainSafe/fil-actor-states
[Upstream] deal_provider_collateral_bounds improvements.
- 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