ProvableHQ / ProvableHQ/aleo-devnode
Make test consensus version heights configurable
Open
Nobody has claimed this yet.
- Dominant language
- Rust
- Stars
- 1
- Forks
- 2
- Avg merge
- 23m
- Merged PRs (30d)
- 1
Description
Follow-up from #13 review: src/start.rs hardcodes test version heights. Consider exposing them via configuration. Acceptable as-is for now.
Review comment: https://github.com/ProvableHQ/aleo-devnode/pull/13#discussion_r3314861991
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by reading the hardcoded test version heights in src/start.rs and the linked review discussion from pull request #13. Determine how those heights should be exposed through configuration, then verify that test consensus version heights can be supplied through that configuration rather than remaining hardcoded.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- rust
- Domain
- testing-qa
- Issue type
- Feature
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Quiet
- Clarity
- Mostly clear
- Newbie friendliness
- 48/100