Nscale revision deploy command now outputs npm warn messages

[shnist]

Hi,

I've recently updated from nscale v0.15 to 0.16. I've noticed now that when I run a nscale rev dep latest command it has started to output npm warn messages:

add www-306968a5$090789a6262c562e44ae0890a6a8eb8af8f24ba1 process
deploying
building
checked out 090789a6262c562e44ae0890a6a8eb8af8f24ba1
npm

WARN package.json karma-coverage@0.2.7 No README data

npm WARN package.json karma-mocha@0.1.10 No README data

npm WARN package.json karma-phantomjs-launcher@0.1.4 No README data

start www-306968a5$090789a6262c562e44ae0890a6a8eb8af8f24ba1 process
starting
> tail -f '/Volumes/Data/Users/fabera/.nscale/log/www-306968a5$090789a6262c562e44ae0890a6a8eb8af8f24ba1.log' # to access the process log

It's not a big thing by any means, but it makes the deployment messages harder to read and to locate the tail command to grab the log files. Was this a deliberate change to the way the messages are outputted when running a deployment?

[pelger]

Hey,
Will take a look at this. No deliberate change IIRC but will check for you.
Cheers.
Peter.

[mcollina]

The deliberate thing was to run the build at every deploy for process containers. This might be wrong, though. @shnist a nice thing you might want to try is to resolve that warnings, i.e. seeing if there is an updated version of those packages that have the README.

[shnist]

@mcollina @pelger - thanks guys. So just to clarify, in v0.16 it runs an npm installation on every deploy now? And this wasn't the case in 0.15?

I'll take a look to see if updated versions of the packages have READMEs.

Cheers

[mcollina]

Only for process-containers. Not sure why it was added, @pelger can you please clarify?

[shnist]

hi @pelger - any update on this?

cheers