Page MenuHomeSoftware Heritage

Don't set $DJANGO_SETTINGS_MODULE in manage.py
ClosedPublic

Authored by vlorentz on Aug 13 2019, 3:56 PM.

Details

Summary

Unlike what the doc says, it overrides --settings.

Required by D1840

Diff Detail

Repository
rDWAPPS Web applications
Branch
DJANGO_SETTINGS_MODULE
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 7290
Build 10323: Cypress tests for swh-web diffsJenkins
Build 10322: tox-on-jenkinsJenkins
Build 10321: arc lint + arc unit

Event Timeline

olasd requested changes to this revision.Aug 19 2019, 2:00 PM
olasd added a subscriber: olasd.

Doesn't this need a change in tox.ini to pull the proper DJANGO_SETTINGS_MODULE?

Makefile.local
27–28

Missing space

This revision now requires changes to proceed.Aug 19 2019, 2:00 PM

Doesn't this need a change in tox.ini to pull the proper DJANGO_SETTINGS_MODULE?

tox.ini invokes pytest, not manage.py, so it does not run this code.

Makefile.local
27–28

Good catch

Nevermind my last question, DJANGO_SETTINGS_MODULE is properly set in pytest.ini (and I'm apparently blind)

This revision is now accepted and ready to land.Aug 19 2019, 2:07 PM
This revision was landed with ongoing or failed builds.Aug 19 2019, 2:08 PM
This revision was automatically updated to reflect the committed changes.