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

Proposal: feat: Enable and autoinstall shell completions for gp CLI #13887

Closed
wants to merge 1 commit into from

Conversation

axonasif
Copy link
Member

@axonasif axonasif commented Oct 15, 2022

Description

Glad I stumbled upon the codebase, I was literally about to write the completion scripts for use with my fish shell. It's kind of funny how this wasn't mentioned anywhere how to setup for bash, which was already there 😬
This updates the completion command and autoinstalls them for different shells on workspace start.

fish
Screenshot 2022-10-16 at 12 50 17 AM
bash
Screenshot 2022-10-16 at 12 52 41 AM
zsh
Screenshot 2022-10-16 at 12 55 22 AM

Related Issue(s)

Fixes #

How to test

  1. Open a workspace in preview env
  2. Type gp and hit Tab key.
  • To check for other shells, switch to them by running fish or zsh
  • And redo 2.

Release Notes

feat: Enable and autoinstall shell completions for gp CLI

Documentation

Werft options:

  • /werft with-local-preview
    If enabled this will build install/preview
  • /werft with-preview
  • /werft with-integration-tests=all
    Valid options are all, workspace, webapp, ide

@werft-gitpod-dev-com
Copy link

started the job as gitpod-build-axonasif-gp-completion.1 because the annotations in the pull request description changed
(with .werft/ from main)

@axonasif axonasif marked this pull request as draft October 15, 2022 13:23
@axonasif axonasif changed the title feat: Enable and autoinstall shell completions for gp CLI Proposal: feat: Enable and autoinstall shell completions for gp CLI Oct 15, 2022
@axonasif
Copy link
Member Author

Had to recreate the PR as werft was failing to boot a VM on an old PR.

@axonasif
Copy link
Member Author

axonasif commented Oct 15, 2022

/werft run

👍 started the job as gitpod-build-axonasif-gp-completion.2
(with .werft/ from main)

@axonasif
Copy link
Member Author

axonasif commented Oct 15, 2022

/werft run with-clean-slate-deployment

👍 started the job as gitpod-build-axonasif-gp-completion.3
(with .werft/ from main)

@axonasif
Copy link
Member Author

axonasif commented Oct 15, 2022

/werft run with-clean-slate-deployment

👍 started the job as gitpod-build-axonasif-gp-completion.7
(with .werft/ from main)

@axonasif axonasif marked this pull request as ready for review October 15, 2022 15:38
@axonasif axonasif force-pushed the axonasif/gp_completion branch 3 times, most recently from e96afc9 to 4d60569 Compare October 16, 2022 13:20
@axonasif
Copy link
Member Author

axonasif commented Oct 16, 2022

/werft run with-clean-slate-deployment

👍 started the job as gitpod-build-axonasif-gp-completion.11
(with .werft/ from main)

@axonasif axonasif added the priority: low (not urgent / proposal) For issues that are proposals and are not directly urgent. Signals no review immediately required. label Oct 16, 2022
@axonasif
Copy link
Member Author

axonasif commented Oct 28, 2022

/werft run with-clean-slate-deployment

👍 started the job as gitpod-build-axonasif-gp-completion.12
(with .werft/ from main)

@axonasif
Copy link
Member Author

axonasif commented Oct 28, 2022

/werft run with-clean-slate-deployment

👍 started the job as gitpod-build-axonasif-gp-completion.13
(with .werft/ from main)

@axonasif axonasif closed this Oct 28, 2022
@axonasif axonasif deleted the axonasif/gp_completion branch October 28, 2022 13:43
@axonasif axonasif restored the axonasif/gp_completion branch October 28, 2022 13:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority: low (not urgent / proposal) For issues that are proposals and are not directly urgent. Signals no review immediately required. release-note size/L team: IDE
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants