Skip to content

ci: use junit annotation action - #9

Draft
G-Rath wants to merge 4 commits into
mainfrom
use-junit
Draft

G-Rath wants to merge 4 commits into
mainfrom
use-junit

Conversation

@G-Rath

@G-Rath G-Rath commented Sep 3, 2023

Copy link
Copy Markdown
Contributor

No description provided.

@eoinkelly

Copy link
Copy Markdown
Contributor

Thanks for making this - it really helps to visualise. Am I missing something in the UI that would let me know which of the test runs in the matrix a given annotation is from?

@eoinkelly

Copy link
Copy Markdown
Contributor

It is nice having the failing test output inline in the diff

@G-Rath

G-Rath commented Sep 3, 2023

Copy link
Copy Markdown
Contributor Author

Am I missing something in the UI that would let me know which of the test runs in the matrix a given annotation is from?

I don't think you can, but that might be improvable with configuration - though also matrixes and annotations have always been a bit lacking in their interaction - namely last I knew they didn't deduplicate annotations but that might have improved; so I think we can probably at least have reports with different names which would make it more obvious, but that might mean getting the same annotation multiple times.

I'll play around with it later this week.

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