-
Notifications
You must be signed in to change notification settings - Fork 118
Pylint should not complain of django constants #41
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
Comments
Good point! If you're interested in making a pull request, the basic idea is in this file: Otherwise I'll do it myself but I probably don't have much time for a few weeks. |
Actually, this is already present: https://github.com/landscapeio/pylint-django/blob/master/pylint_django/augmentations/__init__.py#L246 (insert my confused face) |
You're right, prospector wasn't detecting Django and not loading pylint-django... |
Interesting. I'm using pylint with pylint-django, and pylint still complains about the usual lower-case constants used in Django (such as I'm using Python 3.5.2, Django 1.10.4, pylint 1.6.4, astroid 1.4.9 and pylint-django 0.7.2. PyLint is run via tox 2.5.0. PyLint shows the pylint-django plugin loaded when running What's wrong? |
I am having the same issue. I can see that pylint-django works, since I don't see warnings about |
+1 |
in urls.py I get:
pylint: invalid-name / Invalid constant name "urlpatterns"
When using
register = template.Library()
for a template tag I get:pylint: invalid-name / Invalid constant name "register"
I know I can add comment in code to ignore this but it would be better if support would be built in pylint.
The text was updated successfully, but these errors were encountered: