-
Notifications
You must be signed in to change notification settings - Fork 357
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
chore: lengthen abbreviation to avoid ambiguity #9733
Conversation
✅ Deploy Preview for determined-ui canceled.
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #9733 +/- ##
==========================================
- Coverage 53.44% 53.44% -0.01%
==========================================
Files 1257 1257
Lines 154350 154350
Branches 3298 3298
==========================================
- Hits 82494 82492 -2
- Misses 71706 71708 +2
Partials 150 150
Flags with carried forward coverage won't be shown. Click here to find out more.
|
98bb16d
to
03129fb
Compare
test-react is failing for unrelated reasons - a lot of people are seeing it and it's being worked on in the CI channel. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
(cherry picked from commit dc12336)
Description
First time using the
det resources
CLI and I think the help text could be improved. One quick thing to squeeze in before release is a new command that gets abbreviated as "all", which I think could be confusing as it's not like an option to return all allocations, which is kinda how it reads.Test Plan
Going to install and manually verify
Checklist
docs/release-notes/
See Release Note for details.