8000 Handle empty hook files by hackebrot · Pull Request #862 · cookiecutter/cookiecutter · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Handle empty hook files #862

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 6 commits into from
Dec 8, 2016
Merged

Handle empty hook files #862

merged 6 commits into from
Dec 8, 2016

Conversation

hackebrot
Copy link
Member
@hackebrot hackebrot commented Dec 2, 2016 •
edited
Loading

Address code review comments of #729 and use @pytest.skipif.

⬆️ @michaeljoseph @jcarbaugh

Resolves #632

@hackebrot hackebrot added enhancement This issue/PR relates to a feature request. high-priority needs-merge labels Dec 2, 2016
@hackebrot hackebrot added this to the 1.5.0 Alfajor milestone Dec 2, 2016
@hackebrot hackebrot mentioned this pull request Dec 2, 2016
Closed
10 tasks
@codecov-io
Copy link

Current coverage is 100% (diff: 100%)

Merging #862 into master will not change coverage

@@           master   #862   diff @@
====================================
  Files          16     16          
  Lines         661    667     +6   
  Methods         0      0          
  Messages        0      0          
  Branches        0      0          
====================================
+ Hits          661    667     +6   
  Misses          0      0          
  Partials        0      0

Powered by Codecov. Last update 03dc984...457dc72

@hackebrot hackebrot merged commit 94f1634 into cookiecutter:master Dec 8, 2016
@hackebrot hackebrot deleted the handle-empty-hook-files branch December 8, 2016 22:50
hackebrot added a commit that referenced this pull request Dec 8, 2016
@hackebrot
Update HISTORY.rst for #632, #729 and #862
ce2b52f
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@michaeljoseph michaeljoseph michaeljoseph approved these changes

Assignees
No one assigned
Labels
enhancement This issue/PR relates to a feature request. high-priority
Projects
None yet
Milestone
1.5.0 Alfajor
Development

Successfully merging this pull request may close these issues.
4 participants
@hackebrot @codecov-io @michaeljoseph @jcarbaugh
0