Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixed dependency warning issues #2

Closed
wants to merge 17 commits into from
Closed

Fixed dependency warning issues #2

wants to merge 17 commits into from

Conversation

raykao
Copy link

@raykao raykao commented Jun 13, 2017

  1. remove jade and update to pug
npm uninstall --save jade 
npm install --save pug

This will remove depreciation warning for jade and transformers. The issue being that jade used an older version named transformer and pug now uses jstransformer...so two birds one stone with the update to pug from jade.

  1. Minimatch error is an issue with the png-file-stream dependency on glob-stream which depends on an old version of minimatch. You can get rid of the error by manually changing the package.json for glob-stream to point to the newer 3.0.2 version...but don't bother unless it's completely necessary. This should be an issue flagged with the repo for either or both glob-stream and/or png-file-stream to update their dependencies (package.json)

  2. to further update jade to pug you'll need to update:

edit server.js line 15:

app.set('view engine', 'jade');

to

app.set('view engine', 'pug');

will need to rename file extensions in folder views/ from *.jade to *.pug

in bash you can run the following command in the views folder:

rename "s/jade/pug/" *.jade

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants