Closed ClementTsang closed 5 months ago
Attention: Patch coverage is 98.72881%
with 3 lines
in your changes are missing coverage. Please review.
Project coverage is 55.82%. Comparing base (
d341dbb
) to head (18f20a7
). Report is 2 commits behind head on main.:exclamation: Current head 18f20a7 differs from pull request most recent head 6be7fb7. Consider uploading reports for the commit 6be7fb7 to get more accurate results
Files | Patch % | Lines |
---|---|---|
src/app.rs | 0.00% | 2 Missing :warning: |
src/canvas/widgets/cpu_graph.rs | 50.00% | 1 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Description
A description of the change, what it does, and why it was made. If relevant (such as any change that modifies the UI), please provide screenshots of the changes:
Goes through and updates a few arg/config names to make more sense, and give the docs/arg descriptions a pass.
Something to do in a future PR is to figure out how we're doing json schema rollouts...
Issue
If applicable, what issue does this address?
Closes: #
Testing
If relevant, please state how this was tested. All changes must be tested to work:
If this is a code change, please also indicate which platforms were tested:
Checklist
If relevant, ensure the following have been met:
cargo fmt
)README.md
, help menu, doc pages, etc.)