[eas-cli] improve wording for additional usage - #4094
Conversation
There was a problem hiding this comment.
Claude Code Review
This repository is configured for manual code reviews. Comment @claude review for a one-time review, or @claude review always to subscribe this PR to a review on every future push.
Tip: disable this comment in your organization's Code Review settings.
|
Subscribed to pull request
Generated by CodeMention Warning: The preamble and epilogue options in commentConfiguration are deprecated. Use template instead. |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #4094 +/- ##
==========================================
+ Coverage 61.93% 62.11% +0.18%
==========================================
Files 990 990
Lines 44447 44447
Branches 9340 9340
==========================================
+ Hits 27524 27602 +78
+ Misses 15471 15403 -68
+ Partials 1452 1442 -10 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
🤖 AI code reviewDecision: Approve with comments One lint-related warning remains in the updated usage test: the eslint disable comment does not cover the control-character regex it is meant to suppress. 🟡 Warning (1)
This review is advisory — it never blocks a merge and never auto-approves. |
|
✅ Thank you for adding the changelog entry! |
Why
Currently we print our information about extra use as "overages" which is misleading.
https://exponent-internal.slack.com/archives/C02123T524U/p1784989673512749
How
Replace "overages" -> "extra use" in the usage command and in the pre-build usage printout.
eas build(warning shown before starting a build, when over included credits)Before
After
eas account:usageBefore
After
Test Plan
Updated / added tests.