search

trailsjs / trailpack-hapi

:package: Hapi.js Trailpack
MIT License
19 stars 9 forks source link

added pug as dev dependency #102

Closed weyj4 closed 8 years ago

jaumard commented 8 years ago

Why keep it if it's not use at all on code ?

weyj4 commented 8 years ago

why is it in express?

jaumard commented 8 years ago

For unit testing about rendering views https://github.com/trailsjs/trailpack-express/blob/master/test/app.js#L122 and https://github.com/trailsjs/trailpack-express/blob/master/test/api/controllers/ViewController.js#L6

weyj4 commented 8 years ago

It looks like the only package that requires jade is mocha. So yeah, I'll get rid of pug here but not sure what I can do to fix the jade warning