-
Notifications
You must be signed in to change notification settings - Fork 947
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
[SDK/CLI] added autocompletion for pf cli #2534
Conversation
Hello @wangchao1230 I created this PR as a draft related to #2446 Thank you and have a nice day |
I think we are on the right track. However we are doing a major refactor to layering our sdk into several packages recently. And which has caused some conflict to your PR.
|
src/promptflow/promptflow/_cli/_completers/_param_completers.py
Outdated
Show resolved
Hide resolved
src/promptflow/promptflow/_cli/_completers/_param_completers.py
Outdated
Show resolved
Hide resolved
@microsoft-github-policy-service agree |
e9bf0f4
to
2d1e2a0
Compare
Hello @wangchao1230 I have rebased the MR and added some description per your comments. @YingChen1996 can you please help me with how the pf.autocomplete.sh needs to be sourced ? Thank you for your help |
Hi, @sashokbg .
|
Hello @YingChen1996 ok I will take care of the rest. |
6ab96ac
to
fff6dd7
Compare
fff6dd7
to
dcebfd9
Compare
Hello @YingChen1996 @wangchao1230 I updated my pull request to "ready for review". |
Thanks @sashokbg for your contribution! I have merged the PR. |
Description
This pull request adds shell (bash / zsh) autocompletion for the "pf" cli tool.
It is based on "argcomplete" library https://kislyuk.github.io/argcomplete/ and inspired by "az" cli
All Promptflow Contribution checklist:
General Guidelines and Best Practices
Testing Guidelines