Skip to content

Remove Test Reports step from workflow - #3413

Merged
pjfanning merged 1 commit into
mainfrom
pjfanning-patch-1
Aug 6, 2026
Merged

Remove Test Reports step from workflow#3413
pjfanning merged 1 commit into
mainfrom
pjfanning-patch-1

Conversation

@pjfanning

Copy link
Copy Markdown
Member

Removed Test Reports step from timing-tests workflow.

Our timing tests plan is failing every night. https://github.com/apache/pekko/actions/workflows/timing-tests.yml

This action is now not allowed due to apache/infrastructure-actions#828

Can't find any of the replacements in this allowed list:
https://github.com/apache/infrastructure-actions/blob/main/actions.yml

I don't think this action adds much anyway. When the tests fail, I just look at the logs to find the stacktraces and test reports.

Removed Test Reports step from timing-tests workflow.

@raboof raboof left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I don't think this action adds much anyway. When the tests fail, I just look at the logs to find the stacktraces and test reports.

I'd like seeing test failure details more easily in theory, but in practice I agree I haven't found much use for this particular step

@pjfanning
pjfanning merged commit 75d6a6b into main Aug 6, 2026
10 checks passed
@pjfanning
pjfanning deleted the pjfanning-patch-1 branch August 6, 2026 07:17
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