Skip to content

#59 fix test django settings #60

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

Merged
merged 2 commits into from
Jan 15, 2017
Merged

#59 fix test django settings #60

merged 2 commits into from
Jan 15, 2017

Conversation

timothyjlaurent
Copy link
Contributor

For some reason the [test:pytest] block wasnt working to load the DJANG_SETTINGS_MODULE this adds a pytest.ini file to hold that environment variable.

@coveralls
Copy link

coveralls commented Nov 16, 2016

Coverage Status

Coverage remained the same at 92.364% when pulling f885903 on timothyjlaurent:#59-fix-test-django-settings into d8567f1 on graphql-python:master.

@nickhudkins
Copy link
Contributor

Hey @timothyjlaurent if you merge master back into your branch, we can get travis to stop yelling at us, and perhaps land this PR? I'd like to avoid creating a bunch of PRs where I am just duplicating peoples PRs with master merged back in, hence the reason I am asking you 👍

@syrusakbary
Copy link
Member

Merging the PR (will revert if master goes red).

@syrusakbary syrusakbary merged commit 5d6c7f2 into graphql-python:master Jan 15, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants