Add new simplify command - #1661
Conversation
There was a problem hiding this comment.
Code Review
This pull request introduces a new simplify command for the Ramble framework, designed to identify and optionally remove unused variables, inputs, executables, and compilers from definition files, as well as detect broken references. The review feedback highlights several areas for improvement, including replacing broad exception handling with specific exceptions to improve debugging, optimizing the instantiation of the Keywords class to avoid redundant work, and refactoring duplicated cache-resetting logic in the test suite into a reusable helper function.
Ramble Performance Test MetricsResults produced with commit: c30d974
|
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #1661 +/- ##
===========================================
+ Coverage 93.33% 93.36% +0.03%
===========================================
Files 364 366 +2
Lines 35481 36502 +1021
===========================================
+ Hits 33116 34080 +964
- Misses 2365 2422 +57 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
7d03ee3 to
86ca51f
Compare
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
No description provided.