oxidecomputer / oxidecomputer/humility
Consolidate PMBus debug printing
Open
Nobody has claimed this yet.
- Dominant language
- Rust
- Stars
- 584
- Forks
- 63
- Avg merge
- 3h
- Merged PRs (30d)
- 2
Description
humility pmbus has lovely verbose printing of PMBus data.
However, it's tightly coupled and hard to reuse, so we couldn't reuse it for humility ibc black-box (#325).
We should decouple the pretty-printing and reuse it!
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 tracing the humility pmbus command to find where its verbose PMBus formatting is coupled to command handling, then compare the reuse needed by humility ibc black-box in issue #325. Done means the pretty-printing can be shared by both commands without changing their intended output.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- rust
- Domain
- cli, tooling
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 35/100