Remove sandbox command
TL;DR
The sandbox command has already been deprecated for a while; we can remove it now.
Type
- [ ] Bug Fix
- [ ] Feature
- [ ] Plugin
Are all requirements met?
- [ ] Code completed
- [ ] Smoke tested
- [ ] Unit tests added
- [ ] Code documentation added
- [ ] Any pending items have an associated Issue
Complete description
How did you fix the bug, make the feature etc. Link to any design docs etc
Tracking Issue
NA
Follow-up issue
NA
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:
Comparison is base (
9c87165) 67.69% compared to head (659723f) 67.66%.
Additional details and impacted files
@@ Coverage Diff @@
## master #462 +/- ##
==========================================
- Coverage 67.69% 67.66% -0.03%
==========================================
Files 148 143 -5
Lines 6639 6575 -64
==========================================
- Hits 4494 4449 -45
+ Misses 1855 1841 -14
+ Partials 290 285 -5
| Flag | Coverage Δ | |
|---|---|---|
| unittests | 67.66% <ø> (-0.03%) |
:arrow_down: |
Flags with carried forward coverage won't be shown. Click here to find out more.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
@pingsutw We should probably move the references in flyte repo before removing this command from flytectl https://github.com/search?q=repo%3Aflyteorg%2Fflyte%20%22flytectl%20sandbox%22&type=code
https://github.com/flyteorg/flyte/pull/5034