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: error if no tests reports found #807

Merged
merged 1 commit into from
Jan 2, 2024

Conversation

nikitka
Copy link
Collaborator

@nikitka nikitka commented Dec 29, 2023

Show error about test report absence in the PR comment and job summary.

image

@nikitka nikitka changed the title ci: warn if no tests reports found ci: error if no tests reports found Dec 29, 2023
Copy link

github-actions bot commented Dec 29, 2023

Note

This is an automated comment that will be appended during run.

🔴 linux-x86_64-release-asan: some tests FAILED for commit ed504f5.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
15320 15202 0 31 82 5

🔴 linux-x86_64-relwithdebinfo: some tests FAILED for commit ed504f5.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
59191 49978 0 15 9187 11

@nikitka nikitka merged commit 6ed240e into ydb-platform:main Jan 2, 2024
2 of 4 checks passed
@nikitka nikitka deleted the warn-no-tests branch January 2, 2024 10:25
This was referenced Jan 11, 2024
@pavelvelikhov pavelvelikhov mentioned this pull request Feb 3, 2024
@vitstn vitstn mentioned this pull request Feb 16, 2024
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