move from jade to pug

[tangollama]

Expected behavior:
npm install with no warning

Actual behavior:
npm WARN deprecated jade@1.11.0: Jade has been renamed to pug, please install the latest version of pug instead of jade

Steps to reproduce:
checkout the project
navigate to the frontend directory
npm install

Screenshots (if applicable):
N/A

OS and Browser:

[satnamiaman]

Got these warning while npm install.
Attaching screenshot

[baoqchau]

Hi @tangollama, are we still using jade (or pug) in our project? Because I cannot find jade package in package.json and as I know, HospitalRun using Handlebars as view engine, not Jade

[guillaume-ro-fr]

Hi @baoqchau !

Here the dependancies tree :

hospitalrun@1.0.0-beta /Users/guillaume/Desktop/hospitalrun-frontend
└─┬ electron-prebuilt-compile@1.6.2
  └─┬ electron-compilers@5.9.0
    └── jade@1.11.0

[jbryson3]

I do not get this warning on MacOS with npm 5.5.1 (node 8.9.3)

The only warning I see is:
npm WARN The package electron-protocol-serve is included as both a dev and production dependency.

[baldyjohnmichael]

Mind if I work on this?

[baoqchau]

@baldyjohnmichael go ahead

[stukalin]

@baldyjohnmichael sure you can take over this issue. However, the problem is (sorry if I say obvious things here) that you need to go down the dependency tree and replace jade in the electron-compilers. Actually there are a couple of PRs there, but they were not merged. If you were able to push them, you'd have the issue half-done.
electron-userland/electron-compilers#34
electron-userland/electron-compilers#66

[stale]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.