Skip to content
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

Closed
systemcrash opened this issue Mar 25, 2023 · 5 comments
Closed

request: rename all providers to match real world #2233

systemcrash opened this issue Mar 25, 2023 · 5 comments

Comments

@systemcrash
Copy link
Contributor

Working on PR #2197 raised the question.

I prefer snake_case for the naming.

@tlimoncelli
Copy link
Contributor

rename them where? Rename the identifier ("GCLOUD", "ROUTE53"), the directory (providers/gcloud), or the documentation TOC entries?

@systemcrash
Copy link
Contributor Author

systemcrash commented Mar 26, 2023 via email

@tlimoncelli
Copy link
Contributor

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.

@systemcrash
Copy link
Contributor Author

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.

@tlimoncelli
Copy link
Contributor

I apologize for the abrupt issue closure. That was rude of me.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants