-
-
Notifications
You must be signed in to change notification settings - Fork 734
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
docs: adds Helm docs for dockerconfigjson template #1307
docs: adds Helm docs for dockerconfigjson template #1307
Conversation
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.
Greetings!
Thank you for contributing to this project!
If this is your first time contributing, please make
sure to read the Developer and Contributing Process guides.
Please also mind and follow our Code of Conduct.
Useful commands:
make fmt
: Formats the codemake check-diff
: Ensures the branch is cleanmake reviewable
: Ensures a PR is ready for review
👇 Click on the image for a new way to code review
Legend |
Hi @cebidhem ! thanks for your contribution |
I think this can be of great value to our doc! Just a little nit as this is just one of the possible ways to do it. Also, I think referencing the general issue can be a good idea 😄 |
Thanks for the tips in the review. I made the suggested changes, but I'm trying to figure out why this second commit isn't verified while the first was :) Edit: commit is now verified, I don't what happened with the git.user .... sorry about that |
I also have no idea! Did you commit using GitHub UI? |
Oh I know why, when using |
I think you can |
Or you can just Squash it :) |
Amends with review suggestions Squashed
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
Indeed it was the make docs when I was testing it. Squashed everything! Thanks for taking the time! |
/approve |
/merge |
1 similar comment
/merge |
Hey there,
I'd like to propose this PR to put in the docs what has been discussed in the issue #972.
Indeed, Helm convert the example as nil and @moolen gave a solution in this comment #972 (comment) that should be written in the docs in my opinion.
Thanks for reviewing.