refactor(analysis): rename GVN state constructor#129
Conversation
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## issue-124-gvn-decl-order #129 +/- ##
============================================================
+ Coverage 46.79% 46.80% +0.01%
============================================================
Files 77 77
Lines 17340 17340
============================================================
+ Hits 8114 8116 +2
+ Misses 8307 8306 -1
+ Partials 919 918 -1 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
Summary
newGNumberinghelper tonewNumberinganalysis/gvn.goinstead of repeating the GVN/file contextStacked on #127 so the diff stays limited to the naming convention follow-up.
Testing
Not run locally; repository checkout/test execution was not available in this environment. Verified with compare that this branch only changes two call/definition lines in
analysis/gvn.go.