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

pnpm --recursive --stream is unnecessarily verbose #7228

Open
1 task
FFdhorkin opened this issue Oct 20, 2023 · 1 comment
Open
1 task

pnpm --recursive --stream is unnecessarily verbose #7228

FFdhorkin opened this issue Oct 20, 2023 · 1 comment

Comments

@FFdhorkin
Copy link

FFdhorkin commented Oct 20, 2023

Contribution

Describe the user story

I have a workspace that includes this script: "test": "pnpm --recursive --stream run test",

When I execute it, EVERY line is prefixed.
image

If I don't use stream, it shows the prefix once, then shows a line down the side, which is MUCH cleaner and wastes less horizontal space, but it doesn't show the full log:
image

Additionally, there appears to be a regression of #2148 ; colors on the test timestamps are lost

Describe the solution you'd like

Either:

Here's a mock-up of how I'd like it to look. Note that I changed the color of the workspace and line from that purple to match the color you see in --stream; so if multiple workspaces get run, each would have a different associated color:
image

Describe the drawbacks of your solution

Some people may like the current verbose prefix styling. I figure using a unique line color for projects will be sufficient for most people, though, as long as they aren't relying on grep. (grep is probably a good argument for this being a new flag)

Describe alternatives you've considered

changing the script from pnpm --recursive --stream run test to echo Running workspace A tests... && pnpm --dir workspaceA run test ; echo running Workspace B tests... && pnpm --dir workspaceB run test

@zkochan
Copy link
Member

zkochan commented Oct 21, 2023

From v8.8.0 there is a --reporter-hide-prefix option. Hence, you can try:

pnpm --recursive --stream --reporter-hide-prefix run test

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

No branches or pull requests

2 participants