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

Capitalize acronym names #413

Closed
wants to merge 1 commit into from
Closed

Capitalize acronym names #413

wants to merge 1 commit into from

Conversation

pradyunsg
Copy link

I feel a little bad inside every time I look at these names. I finally decided to do something about it.

@asottile
Copy link
Member

stylistically I actually want to go the other way and remove all the capitalization from the hook names -- would you like to do that instead? (this better matches the file extension and hook ids)

@pradyunsg
Copy link
Author

I strongly prefer all-caps, since they are acronyms and should be capitalized in human-readable strings.

I think we should use all-caps for consistency with any other textual uses, instead of for-the-computer uses -- The JSON spec uses "JSON" and the TOML spec uses "TOML", in their sentences. I'm pretty sure XML, YAML and others would also be doing the same.


I understand the consistency suggestion you make and that's definitely better than status quo. I'm happy to make that change, if you insist on not doing all caps, and I'll update the PR. Just let me know if my paragraph above persuaded you. :)

@asottile
Copy link
Member

I'm willing to admit I'm wrong, but I'd still like them un-capitalized

while you're at it, I've been meaning to change the name for name-tests-test to enforce test file naming (to make it less confusing when the --django flag is passed) -- if you could make that fix as well as part of this that would be great :)

@pradyunsg
Copy link
Author

I think I'm gonna drop this, since I don't really have much time anytime soon to pick this up again.

I still feel the same as I did when I opened the PR -- the acronyms should be consistently capitalized.

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

Successfully merging this pull request may close these issues.

2 participants