search

krakenjs / kraken-js

An express-based Node.js web application bootstrapping module.
Other
4.94k stars 459 forks source link

Fall back on app.json defaults instead of initialising on them. #82

Closed elldritch closed 10 years ago

elldritch commented 10 years ago

Extend #57 to allow for non-default view engines and preprocessors.

Right now, if I'm using jade and stylus, I still need to install adaro, dustjs-linkedin, dustjs-helpers, and less as dependencies in my package.json because Kraken tries to require all default packages on initialisation. Instead, it should check whether I've configured those settings first before loading defaults. Another possibility is to just set adaro, dustjs-linkedin, dustjs-helpers, and less as dependencies of kraken-js itself and load those.

jeffharrell commented 10 years ago

Closing this out as this is a non-issue now that the .next branch has been merged into master. The rendering engine (and all middleware) has been decoupled from the kraken codebase.

If you're daring, there's an early release candidate published on npm as rc along with generator-kraken. Once it's official it will be updated to latest.