Skip to content
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.

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

ci: use junit annotation action #9

Draft
wants to merge 4 commits into
base: main
Choose a base branch
from
Draft

ci: use junit annotation action #9

wants to merge 4 commits into from

Conversation

G-Rath
Copy link
Contributor

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

No description provided.

@eoinkelly
Copy link
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
Contributor

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

@G-Rath
Copy link
Contributor Author

G-Rath commented Sep 3, 2023

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