Problems with `.renderFileAsync` about jstransformer HOT 7 CLOSED

Because I love (and not only me, ofc) to follow readmes and docs, but if they do not helps I go to the tests, if there's no tests or they are so ugly, Im forced to go to the source which is waste of time, then Im forced to follow the deps chain and etc and etc.

from jstransformer.

Need MOAR proof of #16 ? Please, try run this

var hogan = require('jstransformer')(require('jstransformer-hogan'));
var compiled = hogan.compile('<p>Hello, my name is {{name}}.</p>');
var result = compiled.fn({name: 'foobar'});

console.log(result);

mm?

from jstransformer.

#19 fixes this by the way.

from jstransformer.

@tunnckoCore, fixed: jstransformers/jstransformer-hogan@2ea0763. I think you have a point on #16, but please discuss with @ForbesLindesay more on that topic on #16 before we can adopt a project-wide policy.

@hemanth See above commit. For later contributions please always make compile() self-contained. And when you get a chance please add me as owner and tag and publish v0.0.2. Thanks.

from jstransformer.

@tunnckoCore, fixed: jstransformers/jstransformer-hogan@2ea0763.

yea, just notice it. and #19 still not fixes this

from jstransformer.

Fixing things like this belong in the jstransformer tests. We know the tests for jstransformer are very incomplete, and that's why we haven't caught all the bugs yet. Please stop attempting to test jstransformer in the test suites for other libraries.

from jstransformer.

If you need more convincing, check out where the test coverage is currently lacking, relative to where you found a bug:

https://coveralls.io/builds/2333120/source?filename=index.js

from jstransformer.