Skip to content

action: test-docs workflow should create basic junit report - #1767

Merged
v1v merged 4 commits into
elastic:mainfrom
v1v:fix/empty-workflow-test-resutls
Mar 6, 2023
Merged

action: test-docs workflow should create basic junit report#1767
v1v merged 4 commits into
elastic:mainfrom
v1v:fix/empty-workflow-test-resutls

Conversation

@v1v

@v1v v1v commented Mar 6, 2023

Copy link
Copy Markdown
Member

What does this pull request do?

Generate junit test report file with a skipped entry for the test-docs

Why

otherwise, the test-reporter will fail as no artifacts can be found

Issue

elastic/apm-pipeline-library#2121 is the one testing this in isolation.

image

See https://github.com/elastic/apm-pipeline-library/runs/11788165082

v1v added 3 commits March 6, 2023 11:49
otherwise, the test-reporter will fail as no artifacts can be found
@v1v
v1v requested review from a team March 6, 2023 10:57

@beniwohli beniwohli left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Nice workaround! Thanks for the super speedy fix!

Comment thread .github/workflows/test-docs.yml Outdated
Comment thread .github/workflows/test-docs.yml Outdated
@v1v
v1v merged commit e6b807d into elastic:main Mar 6, 2023
@v1v
v1v deleted the fix/empty-workflow-test-resutls branch March 6, 2023 13:56
@v1v v1v mentioned this pull request Mar 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants