Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

AMD support for use with require.js #12

Open
wants to merge 2 commits into
from

Conversation

Projects
None yet
3 participants

tauren commented Jan 11, 2013

Options can now include an amd option, which if true will wrap the compiled templates in an AMD module:

define(function() {
  var amd_jade = ...
  ...
  return amd_jade;
});

Usage:

var app = express()
  .use(express.logger())
  .use(jade_browser('/js/templates.js','**',{
    minify: true,
    amd: true
})

I've checked to make sure it continues to work even if a namespace is specified.

Contributor

ilikebits commented Jan 1, 2014

+1

Contributor

Philmod commented Jan 2, 2014

Hey @tauren ,

I'd like to merge this PR, but there is an issue to merge it automatically. Could you have a look?

Thanks,
Philmod

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment