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

Add test_requirements.txt (fix #377) #945

Merged
merged 1 commit into from May 25, 2017
Merged

Add test_requirements.txt (fix #377) #945

merged 1 commit into from May 25, 2017

Conversation

ramnes
Copy link
Contributor

@ramnes ramnes commented May 15, 2017

Tox is great. Although, a lot of people are used to the simpler pip install -r test_requirements.txt and pytest workflow, including me.

Adding this test_requirements.txt file doesn't cost much and makes things easier for new developers and for rapid testing.

This PR is related to #377.

@codecov-io
Copy link

codecov-io commented May 15, 2017

Codecov Report

Merging #945 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff          @@
##           master   #945   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files          17     17           
  Lines         684    684           
=====================================
  Hits          684    684

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 53ea509...7983c9b. Read the comment docs.

@hackebrot hackebrot added the developers This issue/PR has request to other project developers label May 25, 2017
@hackebrot
Copy link
Member

Hi @ramnes! 👋

Thank you for your contribution. 🍪

@hackebrot hackebrot merged commit 1058d78 into cookiecutter:master May 25, 2017
hackebrot added a commit that referenced this pull request May 25, 2017
@hackebrot
Copy link
Member

FYI I removed -e . from the requirements file, so we can better use it from tox. Otherwise tox would test against the intermediate distribution.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
developers This issue/PR has request to other project developers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants