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
Improve acme CLI options in Let's Encrypt documentation #6762
Conversation
658d9e9
to
31cfcbd
Compare
Hello, Thank you, The CLI flags are case insensitive, so camelCase or lowercase produce the same result. https://docs.traefik.io/operations/cli/#general The blocks of 4 trailing spaces are not errors but used to handle section. |
Hello @ldez, thanks for the feedback. I didn't know about the trailing spaces being used to control section, sorry. Is this still valuable changing the flags to keep standardization across the docs? |
Have an homogeneous documentation is still a valuable change. Could you restore the trailing space, thank you. Could you also edit the PR description. |
31cfcbd
to
6270158
Compare
6270158
to
090cdef
Compare
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 👍
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
What does this PR do?
When reading the current documentation I noticed the Let's Encrypt section shows the acme CLI flags in camelCase format instead of lowercase, which is different from other sections of documentation such as the CLI configuration reference. Because of that, this PR updates the flags to keep homogeneous documentation. Example:
From:
--certificatesResolvers.<name>.acme.caServer
To:
--certificatesresolvers.<name>.acme.caserver
Motivation
My main motivation is to improve the documentation in a way that avoids newcomers get confused.
More