Skip to content
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

feat: add dynamic completion #113

Merged
merged 1 commit into from
Jun 17, 2024
Merged

feat: add dynamic completion #113

merged 1 commit into from
Jun 17, 2024

Conversation

ctrox
Copy link
Contributor

@ctrox ctrox commented Jun 17, 2024

This adds initial support for dynamic completion. For example nctl update application <tab><tab> will return a list of applications in the current project. There are some edge-cases where the arg/flag name does not match the resource name. To make these cases work, they can be added to a map in the predictor.

Copy link
Contributor

@thirdeyenick thirdeyenick left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

This adds initial support for dynamic completion. For example `nctl update
application <tab><tab>` will return a list of applications in the
current project. There are some edge-cases where the arg/flag name does
not match the resource name. To make these cases work, they can be added
to a map in the predictor.
@ctrox ctrox force-pushed the dynamic-completion branch from 52167fa to 5a14a51 Compare June 17, 2024 08:27
@ctrox ctrox merged commit db168e6 into main Jun 17, 2024
2 checks passed
@ctrox ctrox deleted the dynamic-completion branch June 17, 2024 08:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants