Allow requiring ".handlebars" files within Node.js scripts

[jjclark1982]

When using handlebars for server-side templates and storing those templates in individual files, it is convenient to be able to load and compile those files into template functions using require(), like so:

var handlebars = require("handlebars");
var template = require("./template.hbs");
var text = template({name: "Jesse Clark"});

This patch provides a require() handler that loads and compiles .handlebars and .hbs files.

[kpdecker]

This looks awesome!

Could you update the whitespace to match the project standard (2 spaces)? Also any thoughts on unit tests so we make sure that we do not regress this?

[jjclark1982]

I updated the whitespace and added unit tests for both .handlebars and .hbs files to the "Regressions" suite.

[jjclark1982]

The new tests weren't passing in Travis, and I can't think of a way to make this functionality relevant within Ruby, so I moved the tests into a new suite that only runs when require() is available. So they will run from npm test but not from rspec spec.

[kpdecker]

Sounds good.

On Tue, May 14, 2013 at 7:56 PM, Jesse Clark [email protected]:

The new tests weren't passing in Travis, and I can't think of a way to
make this functionality relevant within Ruby, so I moved the tests into a
new suite that only runs when require() is available. So they will run
from npm test but not from rspec spec.

—
Reply to this email directly or view it on GitHubhttps://github.com//pull/515#issuecomment-17914735
.

[kpdecker]

Looks good. Thanks again!

[ishtiaque-khan]

Hey is this thing documented anywhere? Can't seem to find it.