Skip to content

Update CI to output annotations for failures#3411

Closed
cpuguy83 wants to merge 1 commit into
moby:masterfrom
cpuguy83:output_gh_annotations
Closed

Update CI to output annotations for failures#3411
cpuguy83 wants to merge 1 commit into
moby:masterfrom
cpuguy83:output_gh_annotations

Conversation

@cpuguy83
Copy link
Copy Markdown
Member

It is extremely difficult to determine what error occurred in CI. With this change we'll get an annotation with the test log for failed tests.

@cpuguy83 cpuguy83 force-pushed the output_gh_annotations branch 2 times, most recently from 3f236ae to 0bb5ac3 Compare December 14, 2022 20:03
@crazy-max
Copy link
Copy Markdown
Member

crazy-max commented Dec 14, 2022

Might be worth reviving 176faa2 WDYT?

@cpuguy83 cpuguy83 force-pushed the output_gh_annotations branch 2 times, most recently from 835fadb to af71d3b Compare December 14, 2022 21:17
It is extremely difficult to determine what error occurred in CI.
With this change we'll get an annotation with the test log for failed
tests.

Signed-off-by: Brian Goff <cpuguy83@gmail.com>
@cpuguy83
Copy link
Copy Markdown
Member Author

Should this be closed now? @crazy-max

@crazy-max
Copy link
Copy Markdown
Member

Ah right sorry I forgot to close this PR. I don't think we miss anything from the JQ script compared to the TS implementation.

@crazy-max crazy-max closed this Mar 27, 2023
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