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

Tests for custom extensions in hooks #856

Merged
merged 6 commits into from Nov 27, 2016

Conversation

hackebrot
Copy link
Member

@hackebrot hackebrot commented Nov 25, 2016

Adds two tests to replicate #850

Make sure to remove @pytest.mark.xfail when this has been fixed.

@hackebrot hackebrot added the needs-review PR Only: This PR require review from other developer label Nov 25, 2016
@codecov-io
Copy link

Current coverage is 100% (diff: 100%)

Merging #856 into master will not change coverage

@@           master   #856   diff @@
====================================
  Files          16     16          
  Lines         658    658          
  Methods         0      0          
  Messages        0      0          
  Branches        0      0          
====================================
  Hits          658    658          
  Misses          0      0          
  Partials        0      0          

Powered by Codecov. Last update 1bdf78e...67d2938

@hackebrot hackebrot changed the title Custom extensions in hooks Tests for custom extensions in hooks Nov 25, 2016
@hackebrot hackebrot mentioned this pull request Nov 25, 2016
10 tasks
@hackebrot hackebrot added this to the 1.5.0 Alfajor milestone Nov 26, 2016
@michaeljoseph
Copy link
Contributor

lgtm 👍

@hackebrot
Copy link
Member Author

Thank you @michaeljoseph! 😄

@hackebrot hackebrot merged commit 59bfb20 into cookiecutter:master Nov 27, 2016
@hackebrot hackebrot deleted the custom-extensions-in-hooks branch November 27, 2016 23:02
hackebrot added a commit that referenced this pull request Nov 27, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs-review PR Only: This PR require review from other developer
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants