-
Notifications
You must be signed in to change notification settings - Fork 193
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
Pipeline jinja #880
Pipeline jinja #880
Conversation
Codecov Report
@@ Coverage Diff @@
## dev #880 +/- ##
==========================================
+ Coverage 70.52% 70.65% +0.12%
==========================================
Files 33 33
Lines 3651 3673 +22
==========================================
+ Hits 2575 2595 +20
- Misses 1076 1078 +2
Continue to review full report at Codecov.
|
Love the idea |
Okay everything worked fine for me. Run the same commands with this and the old cookiecutter tool.
Will look a bit deeper to be even more on the safe side but so far everything looks good! 👍 |
Approved on Slack ✅ |
Port the pipeline template to use Jinja2 directly instead of cookiecutter, following on from this process for the module template in #878
Makes for cleaner code, nicer file structures (no more file paths with annoying curly braces) and one fewer dependencies.