Yarn: Do not write entire contents of yarn.lock to yarn-error.log

[xPaw]

Do you want to request a feature or report a bug?

Feature?

What is the current behavior?

1. Run anything that produces an error, e.g. yarn thiscommanddoesnotexist
2. yarn-error.log is written with the entire contents of yarn.lock copied to it.

What is the expected behavior?
I'm not sure how useful it is to just copy both package.json and yarn.lock into the error log, especially in the javascript ecosystem where there are an insane amount of (sub-)dependencies.

Please mention your node.js, yarn and operating system version.
Node 9, Yarn 1.5.1, Windows 10.

[hulkish]

+1

[rally25rs]

What does it hurt having it there? I assume the intent is that you could just provide your error log when asking for assistance, instead of 3 files.

[xPaw]

The problem I currently have is that yarn-error.log is created in a somewhat trigger-happy approach, and I haven't really seen many (if at all) issues where the lock file was relevant.

And it puts the stacktrace below the yarn.lock, so you have to scroll down for the actual relevant information.

[hulkish]

I find it really noisy and unhelpful to put the yarn.lock into the error log, personally. This is amplified when using lerna + yarn workspaces, where it creates a yarn-error.log file in every package directory.

Would it be better to allow this to be configurable with an env var?