Skip to content

Feature/161/refactor cmd ctx and scope builders#205

Merged
azriel91 merged 82 commits intomainfrom
feature/161/refactor-cmd-ctx-and-scope-builders
Apr 5, 2025
Merged

Feature/161/refactor cmd ctx and scope builders#205
azriel91 merged 82 commits intomainfrom
feature/161/refactor-cmd-ctx-and-scope-builders

Conversation

@azriel91
Copy link
Copy Markdown
Owner

…and use `miette::Diagnostic` bound for presenting errors.
@codecov
Copy link
Copy Markdown

codecov bot commented Mar 30, 2025

Codecov Report

Attention: Patch coverage is 98.48061% with 38 lines in your changes missing coverage. Please review.

Project coverage is 87.48%. Comparing base (9180f74) to head (4e1d5c4).
Report is 83 commits behind head on main.

Files with missing lines Patch % Lines
crate/cli/src/output/cli_output.rs 87.71% 7 Missing ⚠️
crate/cmd_ctx/src/cmd_ctx_builder_support_multi.rs 97.87% 6 Missing ⚠️
crate/rt_model/src/in_memory_text_output.rs 81.81% 6 Missing ⚠️
crate/cmd_ctx/src/profile_filter_fn.rs 37.50% 5 Missing ⚠️
crate/webi_output/src/webi_output.rs 0.00% 3 Missing ⚠️
crate/rt_model_native/src/storage.rs 89.47% 2 Missing ⚠️
crate/cmd_ctx/src/cmd_ctx_mpnf_params.rs 99.21% 1 Missing ⚠️
crate/cmd_ctx/src/cmd_ctx_mpsf_params.rs 99.61% 1 Missing ⚠️
crate/cmd_ctx/src/cmd_ctx_npnf_params.rs 98.63% 1 Missing ⚠️
crate/cmd_ctx/src/cmd_ctx_spnf_params.rs 99.27% 1 Missing ⚠️
... and 5 more
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #205      +/-   ##
==========================================
- Coverage   88.43%   87.48%   -0.95%     
==========================================
  Files         430      428       -2     
  Lines       45137    41381    -3756     
==========================================
- Hits        39916    36203    -3713     
+ Misses       5221     5178      -43     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@azriel91 azriel91 force-pushed the feature/161/refactor-cmd-ctx-and-scope-builders branch from d1a8ad8 to bcc40e8 Compare April 3, 2025 06:01
@azriel91 azriel91 merged commit dfbba59 into main Apr 5, 2025
13 checks passed
@azriel91 azriel91 deleted the feature/161/refactor-cmd-ctx-and-scope-builders branch April 5, 2025 08:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Refactor Cmd Ctx / Scope Builders

1 participant