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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix flake8 in tests dir #483

Merged
merged 6 commits into from Aug 1, 2015

Conversation

hackebrot
Copy link
Member

This PR fixes all of the flake8 issues that I discovered while working on #480.

Once we got both merged, I will enable flake8 checks for the tests dir in addition to the cookiecutter package in tox.ini. 馃槫

cc @michaeljoseph

@hackebrot hackebrot added the developers This issue/PR has request to other project developers label Jul 30, 2015
@codecov-io
Copy link

Current coverage is 95.64%

Merging #483 into master will not affect coverage as of 78a9e34

@@            master    #483   diff @@
======================================
  Files           12      12       
  Stmts          459     459       
  Branches         0       0       
  Methods          0       0       
======================================
  Hit            439     439       
  Partial          0       0       
  Missed          20      20       

Review entire Coverage Diff


Uncovered Suggestions

  1. +0.87% via cookiecutter/vcs.py#59...62
  2. +0.66% via ...okiecutter/compat.py#78...80
  3. +0.66% via ...okiecutter/compat.py#69...71
  4. See 6 more...

Powered by Codecov

@michaeljoseph
Copy link
Contributor

馃樃 馃憤

pydanny added a commit that referenced this pull request Aug 1, 2015
@pydanny pydanny merged commit 28e5b65 into cookiecutter:master Aug 1, 2015
@hackebrot hackebrot deleted the fix-flake8-in-tests-dir branch August 1, 2015 20:09
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

4 participants