Skip to content
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

Update .gitignore #78

Merged
merged 1 commit into from Jul 2, 2016
Merged

Update .gitignore #78

merged 1 commit into from Jul 2, 2016

Conversation

femtotrader
Copy link
Contributor

@femtotrader femtotrader commented Jul 2, 2016

settings.py shouldn't be in .gitignore because all configuration is in YAML config file or in environment variable

`settings.py` shouldn't be in `.gitignore` because all configuration is in config file or in environment variable
@coveralls
Copy link

coveralls commented Jul 2, 2016

Coverage Status

Changes Unknown when pulling a63db33 on femtotrader:patch-1 into * on mhallsmoore:master*.

@femtotrader
Copy link
Contributor Author

This cause me headhake because unit tests was working locally but not on Travis...
That was only because the right settings.py wasn't push remotely!

So this is VERY important to merge this one

@mhallsmoore mhallsmoore merged commit 83d9c13 into mhallsmoore:master Jul 2, 2016
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.

None yet

3 participants