Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Getting output errors without errors #110

Open
nrfonseca opened this issue Oct 10, 2019 · 4 comments
Open

Getting output errors without errors #110

nrfonseca opened this issue Oct 10, 2019 · 4 comments

Comments

@nrfonseca
Copy link

When I am running NpmCi, NpmRunScript, and NpmPack in an Azure DevOps pipeline, I'm logging errors, here I don't have any errors. For example:

##[error]npm info using [email protected]\n
##[error]npm info using [email protected]
##[error]npm info prepare initializing installer
##[error]npm info prepare Done in 0.015s
##[error]npm http fetch GET 200 https:*** 1096ms
##[error]npm http fetch GET 200 https:*** 1127ms
##[error]npm info extractTree Done in 2.044s
##[error]npm info updateJson Done in 0.015s

@roly445
Copy link

roly445 commented Feb 4, 2020

I can recreate this. Here is a link to my build in pipelines https://deviouscreation.visualstudio.com/stance/_build/results?buildId=188.
I can mitigate the issue by making the clog level as silent.

@nils-a
Copy link
Contributor

nils-a commented Sep 28, 2020

I've seen this, too. However on "manual" runs of npm.
It's "documented" on https://npm.community/t/npm-ci-logs-success-to-stderr/2303:

With all due respect, your build system is doing it wrong. stderr is just the name of a pipe, it doesn’t have any semantic meaning such as “an error occurred in this process”. stderr is where well-behaved CLI programs emit their logging, and the exit code of the process is where a build system should check if there was an error or not (by convention, shell processes exit non-zero when an error occurred).

(Whatever one may think of that comment)

I think this was fixed - do you still have that problem?

@mandalorianbob
Copy link

I still have this problem, which is a bummer. It looks like the change for npm from console.error to console.log was ultimately rejected. The linux users don't care what windows says, Standard Error isn't just for errors.
I'm going to try to mitigate this by using StandardErrorAction to parse out notice vs error.

@mandalorianbob
Copy link

I get this problem when I try to publish to an authenticated feed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants