Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Output GitHub warning/error annotations to stderr #17397

Merged
merged 1 commit into from
May 31, 2024

Conversation

MikeMcQuaid
Copy link
Member

This will mean e.g. opoo etc. will output to stdout and not end up being in the stdout of brew deps etc.

While we're here, remove a duplicate annotation output I noticed in extend/kernel.rb.

Inspired by conversation in:
Homebrew/homebrew-test-bot#1082

This will mean e.g. `opoo` etc. will output to stdout and not end up
being in the stdout of `brew deps` etc.

While we're here, remove a duplicate annotation output I noticed in
`extend/kernel.rb`.

Inspired by conversation in:
Homebrew/homebrew-test-bot#1082
@MikeMcQuaid MikeMcQuaid enabled auto-merge May 31, 2024 08:37
@MikeMcQuaid MikeMcQuaid requested a review from carlocab May 31, 2024 08:37
Copy link
Member

@carlocab carlocab left a comment

Choose a reason for hiding this comment

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

Thanks!

@MikeMcQuaid MikeMcQuaid merged commit fbe4704 into master May 31, 2024
25 checks passed
@MikeMcQuaid MikeMcQuaid deleted the github_annotations_stderr branch May 31, 2024 08:45
@github-actions github-actions bot added the outdated PR was locked due to age label Jul 1, 2024
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jul 1, 2024
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
outdated PR was locked due to age
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants