Remove pylint_django plugin

Open clintonb opened this issue 10 years ago • 3 comments

The pylint_django plugin is required (https://github.com/edx/edx-lint/blob/96f4c58137f73bd4798400c4107dbf1a422dc34f/edx_lint/files/pylintrc#L22), but is not included as a dependency of this package. Unless this is intended only for Django projects, pylint_django should not be a default plugin.

Nov 25 '15 20:11 clintonb

pylint_django is listed in setup.py as a dependency.

I can see why we shouldn't include this plugin in all repos, but I'd like to find a way to uniformly apply it to our Django projects.

Nov 25 '15 22:11 nedbat

It seems I misread setup.py. Closing as there is no immediate need to remove pylint_django.

Dec 02 '15 17:12 clintonb

No need to remove the plugin, but we should disable the message it prints if there's no Django installed: disable=django-not-available

Jan 05 '16 21:01 nedbat