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

Fix replay template name #753

Merged
merged 2 commits into from Jun 15, 2016

Conversation

hackebrot
Copy link
Member

See #752

@hackebrot hackebrot added bug This issue/PR relates to a bug. high-priority needs-review PR Only: This PR require review from other developer labels Jun 9, 2016
@hackebrot
Copy link
Member Author

/cc @eliasdorneles

@codecov-io
Copy link

codecov-io commented Jun 9, 2016

Current coverage is 100%

Merging #753 into master will not change coverage

@@           master   #753   diff @@
====================================
  Files          13     13          
  Lines         583    583          
  Methods         0      0          
  Messages        0      0          
  Branches        0      0          
====================================
  Hits          583    583          
  Misses          0      0          
  Partials        0      0          

Sunburst

Powered by Codecov. Last updated by 36a0080...033e407

@@ -116,7 +116,7 @@ def cookiecutter(
'The repository {0} could not be located.'.format(template)
)

template_name = os.path.basename(template)
template_name = os.path.basename(os.path.abspath(template))
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 looks good to me, thanks @hackebrot !

@michaeljoseph
Copy link
Contributor

👍

@hackebrot hackebrot merged commit 6e4327f into cookiecutter:master Jun 15, 2016
@hackebrot hackebrot deleted the fix-replay-template-name branch June 15, 2016 19:38
hackebrot added a commit that referenced this pull request Jun 15, 2016
terryjbates pushed a commit to terryjbates/cookiecutter that referenced this pull request Jun 17, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug This issue/PR relates to a bug. high-priority 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

4 participants