Hide kubecontext outside of kube project

[matchai]

Description

Add an early return when kube_context is not present.

Motivation and Context

Closes #111

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to not work as expected)

Screenshots (if appropriate):

How Has This Been Tested?

• I have tested using MacOS
• I have tested using Linux

Checklist:

• I have checked that no other PR duplicates mine
• My code follows the code style of this project.
• My change requires a change to the documentation.
• I have updated the documentation accordingly.
• I have updated the tests accordingly.

[Snuggle]

The indentation on this line has gone swimming! Looks good otherwise, please fix before merging. 😊

spacefish/functions/__sf_section_kubecontext.fish

Line 15 in 0b888b0

__sf_util_set_default SPACEFISH_KUBECONTEXT_SUFFIX $SPACEFISH_PROMPT_DEFAULT_SUFFIX

[MindTooth]

Did you notice any changes in rendering speed of the prompt? Felt like this (kubectl)-check caused the shell to halt some before render det new input line.

Testing without Spacefish and using a built in prompt, and it flies away.

Any way for me to completely disable the Kubernetes section, to test the speed out?

[matchai]

This most-recent update should not really affect performance as it's only checking the existence of a variable. We were already performing the kubectl check previously, but not acting on it. 😅

Yes, by setting SPACEFISH_KUBECONTEXT_SHOW to false, the section won't appear, skipping the check.

You can add set -gx SPACEFISH_KUBECONTEXT_SHOW false to your config.fish to disable it.

[MindTooth]

Guess it's all the different checks then. 😄

Thanks.