8000 `@pnpm/log.group` prints `groupstart` CI markers even if `--silent` is used inside GH actions · Issue #9563 · pnpm/pnpm · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

@pnpm/log.group prints groupstart CI markers even if --silent is used inside GH actions #9563

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.

Sign up for GitHub

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

Jump to bottom
Closed
1 of 4 tasks
antitoxic opened this issue May 21, 2025 · 1 comment · Fixed by #9565
Closed
1 of 4 tasks

@pnpm/log.group prints groupstart CI markers even if --silent is used inside GH actions #9563

antitoxic opened this issue May 21, 2025 · 1 comment · Fixed by #9565
Labels
type: bug

Comments

@antitoxic
Copy link
Contributor

Verify latest release

  • I verified that the issue exists in the latest pnpm release

pnpm version

v9.15.9

Which area(s) of pnpm are affected? (leave empty if unsure)

CLI

Link to the code that reproduces this issue or a replay of the bug

No response

Reproduction steps

Just run command using pnpm --silent and workspace concurrency of 1 so that the CI grouping logic kicks in: https://github.com/pnpm/pnpm/blob/main/exec/plugin-commands-script-runners/src/runRecursive.ts#L148C78-L155 in GitHub CI (GH actions)

Describe the Bug

https://github.com/pnpm/components/blob/5a7eefc9f6242e7c5f7d97348592ca87a796a880/log/group/group.ts#L8-L13 will print to stdout even if --silent flag is on

Expected Behavior

No output from pnpm when --silent flag is on

Which Node.js version are you using?

22.6

Which operating systems have you used?

  • macOS
  • Windows
  • Linux

If your OS is a Linux based, which one it is? (Include the version if relevant)

Ubuntu
@antitoxic
Copy link
Contributor Author

Posted link to this issue in the discussion: https://github.com/orgs/pnpm/discussions/3787#discussioncomment-13218200

@antitoxic antitoxic mentioned this issue May 21, 2025
Merged
zkochan pushed a commit that referenced this issue May 22, 2025
@antitoxic
fix: respect silent reporter when deciding whether to output CI group…
b113520 
… stdout markers (#9565)

close #9563
zkochan pushed a commit that referenced this issue Jun 2, 2025
@antitoxic @zkochan
fix: respect silent reporter when deciding whether to output CI group…
55424eb 
… stdout markers (#9565)

close #9563
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
type: bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

ci: respect silent reporter when deciding whether to output CI group stdout markers antitoxic/pnpm
1 participant
@antitoxic
0