Comments (9)
Thanks for merging it so fast!
from gulp-angular-templatecache.
@tandrewnichols, this is interesting. What is your use case where you would want to supply a header and footer later that may not be known at the time you initiate the template cache process?
from gulp-angular-templatecache.
I'm using gulp-remember-cache to only rebuild templates that change and then using gulp-header and gulp-footer to add the wrapper.
from gulp-angular-templatecache.
@tandrewnichols, would a not-so-elegant but functional work-around such as this suffice? I wouldn't be opposed to updating documentation to cover this - respectfully - edge-case.
.pipe(ngtemplates({ header: ' ', footer: ' ' }))
instead of
.pipe(ngtemplates({ header: '', footer: '' }))
from gulp-angular-templatecache.
Um, I guess. Without intending any offense, it seems like a bug, even if it is an edge case, only because it's slightly unexpected behavior. It seems like, if you pass something as the header and footer, those things should be used as the header and footer. The default should be reserved for the case where header and footer aren't supplied. Furthermore, it's a pretty small change right? It's arguably an even smaller (amount of code-wise) change than updating the readme.
All that being said, it's your library, so make what decision you feel is best.
from gulp-angular-templatecache.
@tandrewnichols, no offense taken. It's always good to have varying approaches and opinions on such. I just recently became a collaborator on this project as I worked to remove the gulp-util grab bag package, so I am combing through issues and PRs and seeing what's still applicable. That said, I don't have strong opinions about any particular approach, so long as it's a) not a breaking change, b) covered properly by unit tests, and c) intuitive and/or well-documented.
Would you be so willing to submit a PR for this request, please?
from gulp-angular-templatecache.
I could have a crack at it. Not sure how quickly I'll get to it, but I'll make a TODO so I don't forget.
from gulp-angular-templatecache.
That sounds awesome, Andrew! Looking forward to collaborating on it with you.
from gulp-angular-templatecache.
@tandrewnichols, thanks again for taking charge of this. Appreciate the collaboration.
Simon
from gulp-angular-templatecache.
Related Issues (20)
- Template Body readme instructions result in incorrect string escaping
- Make concatenation an option HOT 1
- Angular Arguments HOT 1
- Crash with a cryptic error message when a folder container a dot HOT 9
- EPERM: operation not permitted HOT 1
- Replace deprecated dependency gulp-util
- Remove deprecated gulp-util by updating gulp-header & gulp-footer HOT 3
- Question: Is it possible to make template urls relative? HOT 8
- Sort output by URI HOT 1
- Fix Security Vulnerabilities HOT 1
- Use ES2015 HOT 4
- Minification is stripping the basic IIFE wrapper from the cached templates HOT 4
- Leading slash added to filenames under gulp 4 HOT 18
- Security Issue: Removing dependency `event-stream` HOT 2
- Error: ENOENT: no such file or directory, open 'target/tmp/templates.js' HOT 1
- Buffer() is deprecated due to security issues.
- Update to NodeJS 8
- npm ERR! Unexpected end of JSON input while parsing near '...0.2"},"dist":{"shasum' HOT 1
- Prototype Pollution security vulnerability HOT 2
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from gulp-angular-templatecache.