-
Notifications
You must be signed in to change notification settings - Fork 319
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
NAS-128601 / 25.04 / Use undefined
instead of EMPTY
#10851
Conversation
undefined
instead of EMPTY
undefined
instead of EMPTY
This is already in 24.10.0. |
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.
Also from the list of changes that needs to be brought from 24.10.1 – there is a check for gpu length that prevents user from removing all gpus from the vm.
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #10851 +/- ##
==========================================
+ Coverage 81.56% 81.92% +0.35%
==========================================
Files 1585 1594 +9
Lines 54136 55589 +1453
Branches 5814 5816 +2
==========================================
+ Hits 44158 45539 +1381
- Misses 9978 10050 +72 ☔ View full report in Codecov by Sentry. |
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.
For fix in .0, I've simply removed the if
condition altogether. If this fix is not correct, then it needs to be fixed in .0 as well.
I do think that we could remove the if
condition though.
This PR has been merged and conversations have been locked. |
Changes:
Minor code improvement
Testing:
Code review is enough