-
Notifications
You must be signed in to change notification settings - Fork 681
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 (get) #5540
base: master
Are you sure you want to change the base?
Conversation
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>
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>
Signed-off-by: zychen5186 <brianchen5197@gmail.com>
Signed-off-by: zychen5186 <brianchen5197@gmail.com>
Signed-off-by: zychen5186 <brianchen5197@gmail.com>
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #5540 +/- ##
==========================================
- Coverage 60.98% 60.98% -0.01%
==========================================
Files 796 796
Lines 51647 51648 +1
==========================================
- Hits 31497 31495 -2
- Misses 17250 17253 +3
Partials 2900 2900
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Signed-off-by: zychen5186 <brianchen5197@gmail.com>
@zychen5186 are you still working on this? |
Hi @davidmirror-ops! No I’m not working on this recently. |
Tracking issue
Related to #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 by typing
!!!Only get method is supported in this PR!!!
Uses GetDomain and ListProjects API to get project and domain if needed.
How was this patch tested?
This feature will only be triggered when -i flag is specified
Case tested:
Setup process
In flytectl directory run
Screenshots
Check all the applicable boxes
Related PRs
#5310
#5357
Docs link