-
-
Notifications
You must be signed in to change notification settings - Fork 90
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
Feature request: django support #33
Comments
Agreed: I'm using django-bootstrap3 which is exclusively imported via template tags and this plugin is not detected at all. Additionally I also use djangocms-admin-style which isn't even imported via tags because it overrides default behaviour simply by being listed in INSTALLED_APPS. This feature would be nice. |
Additional: it also does not detect psycopg2 which is listed in DATABASES. I'd make a pull request if you can tell me where this code should be placed: settings.py is located by manage.py, from there it should simply be parsed for the configuration variables. |
INSTALLED_APPS = [
'name-it-a', 'name-it-b', # pigar: required-packages=package-name-a,package-name-b
'name-it-c', # pigar: required-imports=import-name
] I hope this helps. |
Django doesn't explicitly import, but does use a list of INSTALLED_APPS, which are analogous. Would be great to provide support for that, or at least for a list of additional modules to check. My current work-around is to create an django_imports.py and copy them in, so pigar can pick up the deps.
The text was updated successfully, but these errors were encountered: