alarner / perk

A well documented set of tools for building node web applications.
http://perkframework.com
MIT License
181 stars 31 forks source link

File path display issue #45 #72

Closed hammour closed 7 years ago

hammour commented 7 years ago

When server restarts due to file change, displayed file path should be relative to project root #45

alarner commented 7 years ago

This looks really good. I like the approach of using config.root. One small thing that's a little bit off is that I'm getting output that looks like this:

Server: restarted from file change /lib/config.js
Server: restarted from file change /config/database.js

The / prefix makes me think that it's an absolute path instead of a relative one, which might be confusing. One way to fix this might be to use substr instead of replace like is being done here: https://github.com/alarner/perk/blob/master/build/browserify.js#L51

alarner commented 7 years ago

Look great! 👍