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

Added option to strip template directory. #894

Closed
wants to merge 2 commits into from

Conversation

rmedaer
Copy link

@rmedaer rmedaer commented Jan 27, 2017

The following commit implements a new feature: stripping the template directory.
When you create your project based on cookiecutter template, it's using the first jinja directory as template directory and render it.
Even if you specify an output directory it will always create a sub-rendered-directory.

The strip option skip this first template directory and render the boilerplate in output directory.

Basically this allows to specify the output rendered directory:

cookiecutter --strip -o my-output-directory my-cookiecutter-template

or to render in working directory:

cookiecutter --strip -f -o . my-cookiecutter-template

Raphael Medaer (Escaux) added 2 commits January 27, 2017 12:06
The following commit implements a new feature: stripping the template directory.
When you create your project based on cookiecutter template, it's using the first
jinja directory as template directory and render it.
Even if you specify an output directory it will always create a sub-rendered-directory.

The strip option skip this first template directory and render the boilerplate in
output directory.
In previous commit I added new function/CLI arguments.
This one fix missing arguments in tests.
Pay attention that this will not test strip it self (it's always set to False).
@codecov-io
Copy link

codecov-io commented Jan 27, 2017

Current coverage is 100% (diff: 100%)

Merging #894 into master will not change coverage

@@           master   #894   diff @@
====================================
  Files          17     17          
  Lines         684    685     +1   
  Methods         0      0          
  Messages        0      0          
  Branches        0      0          
====================================
+ Hits          684    685     +1   
  Misses          0      0          
  Partials        0      0          

Powered by Codecov. Last update 04cc38d...6a59a28

@rmedaer
Copy link
Author

rmedaer commented Mar 6, 2017

I gonna close this PR to use a dedicate branch.

@rmedaer rmedaer closed this Mar 6, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants