Skip to content

Conversation

@burmudar
Copy link
Contributor

@burmudar burmudar commented Dec 9, 2025

Print the notice to stderr otherwise agents have to trim the output before passing it to jq

@burmudar burmudar requested a review from a team December 9, 2025 14:52
@burmudar burmudar self-assigned this Dec 9, 2025
Copy link
Member

@keegancsmith keegancsmith left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

remove the notice, we don't advertise the command so it doesn't really add any value I think?

@burmudar
Copy link
Contributor Author

burmudar commented Dec 9, 2025

remove the notice, we don't advertise the command so it doesn't really add any value I think?

yeah valid

not neeed since command isn't advertised so experimental is implied
@burmudar burmudar enabled auto-merge (squash) December 10, 2025 09:17
@burmudar burmudar merged commit ee83af1 into main Dec 10, 2025
8 checks passed
@burmudar burmudar deleted the wb/stderr-notice branch December 10, 2025 09:20
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.

3 participants