-
Notifications
You must be signed in to change notification settings - Fork 539
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
Integrate Bubbletea list CLI for command selecting #5357
base: master
Are you sure you want to change the base?
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #5357 +/- ##
=======================================
Coverage 61.08% 61.09%
=======================================
Files 793 794 +1
Lines 51156 51214 +58
=======================================
+ Hits 31251 31290 +39
- Misses 17033 17043 +10
- Partials 2872 2881 +9
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
fe80a86
to
e1fa9b1
Compare
Signed-off-by: zychen5186 <brianchen5197@gmail.com>
Signed-off-by: zychen5186 <brianchen5197@gmail.com>
Signed-off-by: zychen5186 <brianchen5197@gmail.com>
Signed-off-by: zychen5186 <brianchen5197@gmail.com>
e1fa9b1
to
7901da9
Compare
Signed-off-by: zychen5186 <brianchen5197@gmail.com>
Signed-off-by: zychen5186 <brianchen5197@gmail.com>
Signed-off-by: zychen5186 <brianchen5197@gmail.com>
Tracking issue
Related to https://github.com/flyteorg/flyte/issues/<4440>
Why are the changes needed?
What changes were proposed in this pull request?
User can select the command they want to use through the CLI
How was this patch tested?
This feature will only be triggered when -i flag is specified
Case I tested:
Setup process
Screenshots
Check all the applicable boxes
Related PRs
#5310
Docs link