-
-
Notifications
You must be signed in to change notification settings - Fork 1k
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
Replace aliased imports with their canonical name #1720
Comments
Hmm, I think we're in a difficult place with Renaming I'm totally on board with |
Ah yes, that's right. On second sight I wonder where the
Yeah, I think that's the way to go (if we change anything!) |
Do you mean |
Oh, |
I don't think I care enough; I was very confused and it's so cosmetic as to not matter (much). We can revisit this later if it comes up again but for now I'm going to close this. |
See #1717
When I see a
settings
variable in a Django project I immediately assume that it'sdjango.conf.settings
When I see
dt_settings
I have to look it up.Also,
override
immediately tells medjango.utils.translation.override
. If it's not obvious I sometimes import the module and use the function astranslation.override
but that happens almost never.It's a low priority issue but I think it should be changed if it's possible without breaking too many pull requests :)
The text was updated successfully, but these errors were encountered: