always report project generator string that's used in the console output about projectgenerator HOT 9 CLOSED

yes it was in the console, looks like it's been stripped out. Shall I put that back ?

from projectgenerator.

yeah I think it's a good idea to put it back in, so we can debug easier any problems.

from projectgenerator.

There you go, with a readme entry and I also put back my last forgotten commit from ofZach/pgElectron

from projectgenerator.

thanks ! I don't think we need it in an ever present console -- just to be reported on the console you see with success / failure dialog...

from projectgenerator.

It's pretty discreet like this and I find it useful useful to trace back generated projects and, as you said, look at their command line string. What about simplifying/shortening it so that you only see the command history, clicking on each of them them will show their console output.

The verbose mode setting already defines if we see the success modal with a console (btw, this setting does nothing for the moment... and is not present in the default settings.json)

So I see the verbose option more as a project related setting while the showConsole setting is related to all projects.

from projectgenerator.

no I'm sorry we already decided not to have this console. we already have 3 potential console outputs

a) the console output if you run electron from the command line (ie npm start)
b) the console output if you run developer tools
c) the console output from success / failure on generate project.

I really don't think we need a 4th and I think this just confuses things to have this user facing. all I am asking for here is to always put the command line string that we are firing off the command line tool in the console output of (c). We could improve the reporting in (a / b) if it feels like we need to give better feedback. thanks!

from projectgenerator.

(ps this is where we decided not to have this additional console ofZach/pgElectron#24 (comment))

from projectgenerator.

Ok, that sounds good! Sorry for the confusion...
In fact i saw the showConsole as a hidden setting but by cleaning up the other consoles it's enough too. 👍

I added some commits removing the old console code and adding support for verbose option.

from projectgenerator.

I've added this now -- closing this issue

from projectgenerator.