-
Notifications
You must be signed in to change notification settings - Fork 143
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 private registries integration docs #1237
Conversation
@ArieLevs Thanks for the contribution, would you mind adding a link to the page from the index.md file within docs/use-tf-controller/ directory, so that the page will become visible when the documentation is generated as well? |
@ilithanos done |
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.
Thanks a lot for getting the link from index.md into the PR.
You've written a good piece of documentation and I really appreciate you taking the time to help out.
There's a few corrections needed to increase the readability of the documentation suggested in the review, when these sections are updated I can merge the PR 👍
docs/use-tf-controller/integration-with-terraform-private-registries.md
Outdated
Show resolved
Hide resolved
docs/use-tf-controller/integration-with-terraform-private-registries.md
Outdated
Show resolved
Hide resolved
docs/use-tf-controller/integration-with-terraform-private-registries.md
Outdated
Show resolved
Hide resolved
docs/use-tf-controller/integration-with-terraform-private-registries.md
Outdated
Show resolved
Hide resolved
docs/use-tf-controller/integration-with-terraform-private-registries.md
Outdated
Show resolved
Hide resolved
…stries.md Co-authored-by: Tonni Follmann <[email protected]>
…stries.md Co-authored-by: Tonni Follmann <[email protected]>
…stries.md Co-authored-by: Tonni Follmann <[email protected]>
…stries.md Co-authored-by: Tonni Follmann <[email protected]>
…stries.md Co-authored-by: Tonni Follmann <[email protected]>
I knew I should let chatGPT do the grammer 😅 done, committed via the PR |
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.
Haha :) LGTM 🥇 thanks again for the contribution
fixes: #1189