Skip to content
This repository has been archived by the owner on Mar 26, 2018. It is now read-only.

Mustache precompiling with grunt-contrib-hogan #284

Closed
wants to merge 1 commit into from
Closed

Mustache precompiling with grunt-contrib-hogan #284

wants to merge 1 commit into from

Conversation

erikmajlath
Copy link

grunt-mustache is used only for concatination of templates. I have exchanged grunt-mustache with grunt-contrib-hogan which precompiles templates and also concatinates them into one file.

In this commit I modified

  • gruntfile template
  • _bower.json
  • _package.json
  • app/index.js to add hogan as vendor script
  • view/index.js to include templateFramework variabe while templating so it's possible to output this.template.render which is hogan's way of rendering precompiled template
  • all views to use this.template.render

[MOD]gruntfile hogan task
[MOD]requirejs_app added hogan path
[MOD]app/index.js added vendor script hogan
[MOD]_package.json added grunt-contrib-hogan
[MOD]_bower.json added hogan 3.0.2, contains all builds
[MOD] all views updated with template.render hogan method
[MOD] passing templateFramework in views/index.js so we can decide how to render
@revathskumar
Copy link
Member

@erikmajlath Sorry for late reply. Some how I missed this PR.
will take a look as soon as possible.

Fixes #283

@sindresorhus
Copy link
Member

@erikmajlath Can you fix the merge conflict?

@arthurvr
Copy link
Member

ping @erikmajlath - still interested in taking this up?

@arthurvr
Copy link
Member

arthurvr commented Jul 2, 2015

Closing due to inactivity.

@arthurvr arthurvr closed this Jul 2, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants