Skip to content

👷 Allow skipping benchmark job in test workflow#14974

Merged
tiangolo merged 1 commit intomasterfrom
fix-CI-for-docs-PRs
Feb 23, 2026
Merged

👷 Allow skipping benchmark job in test workflow#14974
tiangolo merged 1 commit intomasterfrom
fix-CI-for-docs-PRs

Conversation

@YuriiMotov
Copy link
Member

@YuriiMotov YuriiMotov commented Feb 23, 2026

The benchmark job is skipped for docs-only PRs, but it's still required in check job. This causes test wokflow to fail for PRs that only modify docs

@codspeed-hq
Copy link

codspeed-hq bot commented Feb 23, 2026

Merging this PR will not alter performance

✅ 20 untouched benchmarks


Comparing fix-CI-for-docs-PRs (ffe7edb) with master (5161f7b)1

Open in CodSpeed

Footnotes

  1. No successful run was found on master (da19374) during the generation of this report, so 5161f7b was used instead as the comparison base. There might be some changes unrelated to this pull request in this report.

Copy link
Member

@tiangolo tiangolo left a comment

Choose a reason for hiding this comment

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

Good catch, thank you! 🙌

@tiangolo tiangolo merged commit 6af3832 into master Feb 23, 2026
32 checks passed
@tiangolo tiangolo deleted the fix-CI-for-docs-PRs branch February 23, 2026 18:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants