-
-
Notifications
You must be signed in to change notification settings - Fork 48
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also, should be concat
to existing ignore, or simply disregard them?
src/Commands/Serve/index.js
Outdated
@@ -164,6 +165,19 @@ class Serve extends Command { | |||
} | |||
|
|||
const nodemon = require('nodemon') | |||
const foldersToIgnore = [ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I believe, we can make it little bit more organized. Also their is no need to add *
to user defined ignore values, since it removes the flexibility of defining custom blobs.
The following feels more approachable to me
['/tmp/*', '/resources/*', '/public/*'].concat(ignore || []).map((folder) => `${process.cwd()}/${folder}`)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also, should be concat
to existing ignore, or simply disregard them?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also, should be concat
to existing ignore, or simply disregard them?
I believe we should keep the default ignored folders and simply concat the value. The requested change has been done. 馃憤 |
up? |
Hey 馃憢
This PR add two things.
Per default we don't watch the folder
resources
, this folder should only contain frontend stuff or compiled assets. There's no need to reboot the whole server when they change.It's adding the
--ignore
flag to theserve
command. It let us set custom path to be ignored bynodemon
.How to use it?
This command will ignore the default ignored folder +
nuxt
,myCustom
andnext
.