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

adding exceptions #13

Open
lrstanley opened this issue Dec 24, 2018 · 5 comments
Open

adding exceptions #13

lrstanley opened this issue Dec 24, 2018 · 5 comments

Comments

@lrstanley
Copy link

Thoughts on adding something similar?:

https://github.com/serenize/snaker/blob/a683aaf2d516deecd70cad0c72e3ca773ecfcef0/snaker.go#L100-L150

This would make it easier to use this for code generation. I.e. something like TEST_HTTP doesn't get converted to TestHttp (which isn't a go standard), rather TestHTTP.

@lrstanley
Copy link
Author

I think this would also address issues like #12 (i.e. convert every capital letter after the first into lowercase, unless it matches an exception). This would convert CUSTOMER_ID into CustomerID if done properly.

@iancoleman
Copy link
Owner

Looks ok.

Only doubt is whether RAM should be on the list for the purpose of this tool.

I'll look into adding this, I think it's a good idea. Thanks for raising the issue.

@danielcooperxyz
Copy link

I can't help but think it should be from the lint code file, and would work as a specific GolangCamelCase method.

@lrstanley
Copy link
Author

Looks like this was partially added in 10b280f quite awhile back, enough for others to be able to specify their own acronyms. Not perfect, but I think it's enough to close this issue.

@lrstanley
Copy link
Author

lrstanley commented Sep 9, 2023

Actually, it looks like that functionality doesn't work as intended. No matter how I try and use ConfigureAcronym.

@lrstanley lrstanley reopened this Sep 9, 2023
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

3 participants