segment-boneyard / component-jade

A plugin to transpile Jade files for the component builder.
12 stars 8 forks source link

var template = require('template.jade'); doesn't work #19

Closed azhang closed 10 years ago

azhang commented 10 years ago

as shown in the example. It only works if you do

 var template = require('template.jade.html');

It's a quick fix, I'll do a pull request by tonight.

Another issue with that same line of code is that by setting file.filename = basename(file.filename)+'.html', it causes problems with equivalently named templates in different folders. It should just be file.filename.

ianstormtaylor commented 10 years ago

ah yeah that is dumb. i agree that template.jade makes the most sense, since that's what goes in the templates array in component.json anyways. will gladly take a pull! thanks for looking into that

azhang commented 10 years ago

Sorry about creating all these issues! I haven't been able to figure out how to link a pull request with an existing issue.

ianstormtaylor commented 10 years ago

no worries, more issues is fine by me :)

azhang commented 10 years ago

Cool, thanks! :)