Watch code and run about v12

[hugojosefson]

@bishoymelek It's merged, and will be released by as [email protected] - via botpress/botpress#187 (comment)

BTW @slvnperron Issue #46 can be closed now too, because it was fixed by PR botpress/botpress#187. I hadn't refreshed this browser tab :D

[rocel]

@slvnperron @eyalcohen4
isn't it what the npm run watch already does? it uses nodemon.
I actually do use this command while testing/hacking botpress (the server or the cli).

[slvnperron]

hey, glad you like botpress :)! It's a very nice idea. What about implementing it with an optional -w or --watch flag on bp start? We would love it if you could make this pull request, it's something I've been wanting too!

[eyalcohen4]

Cool! I'll do it. thinking about add nodemon as a module, and in the start file add it to the function.
What do you say? @slvnperron

[eyalcohen4]

If i understand well, you are talking about botpress development.

I meant when developing the bot instance @rocel

[slvnperron]

@eyalcohen4 that's how I would do it, yes. Though only add it as a devDependencies and make the command available only in development if that makes sense? thanks

[rocel]

oh ok, sorry then. Indeed a flag on start would be user friendly and nodemon a good idea.

[hugojosefson]

I have used https://www.npmjs.com/package/piping before, with great success in server applications.

Maybe that could work here too? Perhaps its use needs to be integrated with, or replace, the current handling of worker process in botpress.start(): https://github.com/botpress/botpress/blob/master/src/botpress.js#L247

[hugojosefson]

Looking into this now.

[hugojosefson]

Implemented. Now waiting for review on botpress/botpress#187, which should fix this.

[bishoymelek-zz]

hi @hugojosefson can you please help me use what you implemented so i can use nodemon with botpress ? is there steps to do or something ? thank you

[hugojosefson]

@bishoymelek Sure, I'll look into it!