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(nats): Add NATS Context #5900
Conversation
I'm not sure what's going on with the print all tests. |
Ah, helps if I read the test output correctly. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have made a few more suggestions, but this is already looking fairly good.
I'm not sure what the config is failing on. I ran |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Thanks for the contribution @hooksie1! |
Description
Adds a module for getting the current NATS context. Uses the nats cli to get the current context name.
Motivation and Context
This is similar to having the Kubernetes context displayed so it's always known which context you are in.
Screenshots (if appropriate):
How Has This Been Tested?
Added the mock nats cli and then created tests to make sure styling matched.
Also built locally and tested changing contexts and styles.
Checklist: