-
Notifications
You must be signed in to change notification settings - Fork 4
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
Add support for jsonpath to the CLI #127
Conversation
1. [Populate the manifests to Public Hub](../example/public-hub-content.mdx#populate-the-manifests-into-hub). | ||
1. (For AWS only) Create a [Type Instance](../example/typeinstances.md#aws-credentials) with credentials. | ||
1. Prepare input file `input.yaml` | ||
2. Copy content of the `generated` directory into `manifests` directory. |
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.
You don't need to enumerate the content with 1,2,3 etc. You can read about it more here.
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.
Changed to the previous version
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.
Approve 👍 also take a look at the comments
Co-authored-by: Mateusz Kuziemko <[email protected]>
</TabItem> | ||
</Tabs> | ||
</div> | ||
import Tabs from '@theme/Tabs'; |
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.
Unfortunately, the numbering started working but now we have the:
import Tabs from '@theme/Tabs'; import TabItem from '@theme/TabItem';
in the view of the Helm website.
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've fixed it
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 🚀
Description
Changes proposed in this pull request:
Related issue(s)
jsonpath
output capact#614