unfold/browserify-hogan

Browserify transform plugin for Hogan.js templates

browserify-hogan's Issues

Add me as NPM maintainer?

Hey @sbrekken, could you add me as a maintainer on NPM too, so I can publish my update?

npm owner add callumlocke browserify-hogan

git push --tags

Hi - would you mind pushing the tags to this repository so that it can be pulled in by package management systems other than npm (eg. bower)?

Use Hogan v3.x

I know that the current Hogan version on npm is still v2.0.0 but since v3.0.0 in the official repository passes all tests I'd like to use Hogan v3.x with browserify and this plugin as well.

https://github.com/twitter/hogan.js

Publish new version to npm

I want to use the html-minifier functionality which was added recently. Can you publish a new version to npm?

cc @abi @jhiesey

Option to include whole Hogan.js library in templates

Comment from @jkrems on issue #2:

Would you be open to making this optional? Not including Hogan quietly disables support for lamdas (e.g. {{#myLamda}}text{{/myLamda}}) which checks for template.c - and since the template itself is not exposed, it's currently impossible to fix from the outside.

Export the whole template object?

@sbrekken do you mind if I make the following change to this transform: I want to change it so the compiled module exports the whole Hogan template object, not just the render function. So usage would change to:

var myTemplate = require('./my-template.mustache');
$('body').append( myTemplate.render({foo: 'bar'} );
                             ^^^^^^

reasons:

• it's more in line with Hogan's docs (they describe a "compiled template" as being an object with a render method, not just a function)
• it would make it easier to inspect compiled template code (for debugging purposes)... Currently if you console.log the template, you just get function (data) {return template.render(data)}, so you have to go looking in the bundled code.

I just wanted to check if you had any objection before I go ahead and do this, as it's effectively an API change (so I would bump the version to 0.1.0).

gulp-browserify

I use gulp-browserify in my build process and I seem to be getting errors when using this.

I require my template like so:

template: require('../Templates/invoiceDetail.ms'),

but when running gulp I get the following error:

/Users/bramdevries/Documents/School/2013-2014 3 Devine/Stage/Madewithlove/kaash/public/app/js/Templates/invoiceDetail.ms:1
<p>Hello</p>
^
ParseError: Unexpected token <
    at wrapWithPluginError (/Users/bramdevries/Documents/School/2013-2014 3 Devine/Stage/Madewithlove/kaash/node_modules/gulp-browserify/index.js:44:10)
    at Stream.<anonymous> (/Users/bramdevries/Documents/School/2013-2014 3 Devine/Stage/Madewithlove/kaash/node_modules/gulp-browserify/index.js:103:28)
    at Stream.<anonymous> (/Users/bramdevries/Documents/School/2013-2014 3 Devine/Stage/Madewithlove/kaash/node_modules/gulp-browserify/node_modules/browserify/index.js:299:22)
    at Stream.EventEmitter.emit (events.js:117:20)
    at Stream.onerror (/Users/bramdevries/Documents/School/2013-2014 3 Devine/Stage/Madewithlove/kaash/node_modules/gulp-browserify/node_modules/browserify/node_modules/stream-combiner/index.js:29:18)
    at Stream.EventEmitter.emit (events.js:117:20)
    at Stream.<anonymous> (/Users/bramdevries/Documents/School/2013-2014 3 Devine/Stage/Madewithlove/kaash/node_modules/gulp-browserify/node_modules/browserify/index.js:527:30)
    at Stream.stream.write (/Users/bramdevries/Documents/School/2013-2014 3 Devine/Stage/Madewithlove/kaash/node_modules/gulp-browserify/node_modules/browserify/node_modules/through/index.js:26:11)
    at Stream.ondata (stream.js:51:26)
    at Stream.EventEmitter.emit (events.js:95:17)