-
Notifications
You must be signed in to change notification settings - Fork 402
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
request: rename all providers to match real world #2233
Comments
rename them where? Rename the identifier ("GCLOUD", "ROUTE53"), the directory (providers/gcloud), or the documentation TOC entries? |
All of the above. At least so everything is synchronized.
I'd considered putting each provider markdown in the provider folder so
there's only 1 source tree for adding a provider, but perhaps this creates
problems when building/copying the docs and new record types would probably
seem like a weird exception if it wasn't handled similarly.
…On Sun, 26 Mar 2023 at 14:57, Tom Limoncelli ***@***.***> wrote:
rename them where? Rename the identifier ("GCLOUD", "ROUTE53"), the
directory (providers/gcloud), or the documentation TOC entries?
—
Reply to this email directly, view it on GitHub
<#2233 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAE6DUPNMI3EABQMK77BITDW6A4KDANCNFSM6AAAAAAWHVCRSE>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
Rejected. This would break all existing configurations for no other reason than "I prefer snake_case for the naming". Please use the issues feature to file bugs and feature requests. This is neither. |
Actually, as expressed in the auto TOC PR, the hope was to have all providers in snake_case so that TOC name printing would be reasonably predictable and elegant. |
I apologize for the abrupt issue closure. That was rude of me. |
Working on PR #2197 raised the question.
I prefer snake_case for the naming.
The text was updated successfully, but these errors were encountered: