diff options
author | Gar <gar+gh@danger.computer> | 2021-08-17 17:33:39 +0300 |
---|---|---|
committer | Gar <gar+gh@danger.computer> | 2021-08-17 19:02:54 +0300 |
commit | 487731cd56a22272c6ff72ef2fa7822368bf63e3 (patch) | |
tree | fb7a847af635ccc34060a49a93a42dcc6da56055 /tap-snapshots | |
parent | 7a582644d544af2c2b450b18bed1b4d5f71cd786 (diff) |
fix(logging): sanitize logged argv
Wraps logged process.argv in `replaceInfo`
Removes logged process.argv from EJSONPARSE warning for top level
package.json merge conflicts. It is currently not even working (er.file
is not being populated by the parsing library right now), and
process.argv contains fullly resolved paths which isn't very nice
looking. The user knows what they typed, it's enough to tell them to
retry.
PR-URL: https://github.com/npm/cli/pull/3658
Credit: @wraithgar
Close: #3658
Reviewed-by: @nlf
Diffstat (limited to 'tap-snapshots')
-rw-r--r-- | tap-snapshots/test/lib/utils/error-message.js.test.cjs | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/tap-snapshots/test/lib/utils/error-message.js.test.cjs b/tap-snapshots/test/lib/utils/error-message.js.test.cjs index 5b6e3c85a..e8f817cd1 100644 --- a/tap-snapshots/test/lib/utils/error-message.js.test.cjs +++ b/tap-snapshots/test/lib/utils/error-message.js.test.cjs @@ -1537,9 +1537,7 @@ Object { String( Merge conflict detected in your package.json. - Please resolve the package.json conflict and retry the command: - - $ arg v + Please resolve the package.json conflict and retry. ), ], ], |