caktus / django-project-template

Django project template for startproject (Requires 2.2+)
211 stars 53 forks source link

remove unneccesary database creation by travis #277

Closed dchukhin closed 7 years ago

dchukhin commented 7 years ago

While working on a different project, travis suddenly began failing because it no longer had the USER variable set. According to travis documentation, we shouldn't rely on this variable.

It doesn't seem necessary (as pointed out by @vkurup ) to create the database in travis right before running tests, since the tests themselves create a test database, so I have removed the line to create a database by travis.

dpoirier commented 7 years ago

Just to be clear - I agree it doesn't seem necessary, but we have verified that, right?

dchukhin commented 7 years ago

Good thinking. I have made a pull request for the website here, which passes travis tests, so I think so.

dpoirier commented 7 years ago

:scissors: