Skip to content

Conversation

nlf
Copy link
Contributor

@nlf nlf commented Aug 8, 2022

when we implemented the temp file for scripts, a subtle thing that we lost was the banner logging both the script itself as well as the arguments it received. this pull request restores that behavior, which i think is more clear and helpful for our users

current behavior (in this repo):

❯ npm test -- --coverage-report=html

> @npmcli/run-script@4.2.0 test
> tap

new behavior:

❯ npm test -- --coverage-report=html

> @npmcli/run-script@4.2.0 test
> tap --coverage-report=html

@nlf nlf requested a review from a team as a code owner August 8, 2022 17:17
@nlf nlf force-pushed the nlf/restore-banner-args branch from e018630 to 090d096 Compare August 8, 2022 17:28
@nlf nlf merged commit 8e08311 into main Aug 9, 2022
@nlf nlf deleted the nlf/restore-banner-args branch August 9, 2022 20:31
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

Successfully merging this pull request may close these issues.

2 participants