Migrating from Jade to pug

andygrunwald commented 3 years ago

This project has a few Security dependencies, see https://github.com/andygrunwald/things-with-buzzers-jeopardy/security/dependabot Most of them are related to Jade (or sub dependencies). Jade itself is deprecated and has been renamed to pug, see Rename from "Jade" chapter at https://www.npmjs.com/package/pug/v/2.0.0

By this, a few breaking changes were introduced. See https://github.com/pugjs/pug/issues/2305

Migrate from Jade to pug. By this, we are able to resolve the security issues.

Remove Jade from packages.json
Add pug ~2.x.x to package.json
Switch app.set('view engine', 'jade'); to app.set('view engine', 'pug');
Rename all *.jade files to *.pug
Migrate all templates to the new syntax (there seems to be a clash with #{ and inline interpolation

dmz38 commented 3 years ago

I'm willing to work on this and will investigate over the next few weeks.

andygrunwald commented 3 years ago

Great, thanks @dmz38!

andygrunwald / things-with-buzzers-jeopardy