-
-
Notifications
You must be signed in to change notification settings - Fork 175
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
Fix creating checks with more than 50 annotations #215
Conversation
This comment has been minimized.
This comment has been minimized.
Unit Test Results (Test Files) 24 files ±0 4 errors 21 suites ±0 39m 12s ⏱️ ±0s For more details on these parsing errors, failures and errors, see this check. Results for commit e9870ab. ± Comparison against base commit 043296c. ♻️ This comment has been updated with latest results. |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
Creating more than 50 annotations works: https://github.com/EnricoMi/publish-unit-test-result-action/runs/4860016192 |
While looking into #212 it turned out that creating checks with more than 50 annotations does actually not work. The action creates multiple checks, which overwrites earlier checks. It does not produces the error reported in #212 though. This might have caused #183.
While testing this it turned out that if there are test names longer than
max_chunk_size
, the action turns into an endless loop. This unlikely case is handled now by dropping all test names (skipping test list annotations entirely) while working otherwise.This fixes creation of checks with more than 50 annotations. This also fixes #214.