Skip to content
This repository was archived by the owner on Sep 17, 2025. It is now read-only.

Conversation

@marckhouzam
Copy link
Member

Completion tests adaptation for helm/helm#7690
Should only be merged once the helm PR is merged.

@marckhouzam
Copy link
Member Author

This PR originally simply added the word fish in the necessary completion tests and adapted the tests for some of the plugin completion changes.

It now also allows to actually test fish completions.
This required a big refactoring but now we test bash, zsh and fish.

Signed-off-by: Marc Khouzam <[email protected]>
This commit moves the creation of kube-context config files to the same
file as where the rest of the configuration is setup.

Signed-off-by: Marc Khouzam <[email protected]>
@marckhouzam marckhouzam marked this pull request as draft April 14, 2020 03:02
@marckhouzam
Copy link
Member Author

I just learned about Draft PRs 😁

This is a major refactor that allows to test the fish shell.
Running the completion tests does not change, but now they include
fish on Linux and on Mac.

Signed-off-by: Marc Khouzam <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants