Thanks to visit codestin.com
Credit goes to github.com

Skip to content

👷 Allow skipping benchmark job in test workflow#14974

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

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

Conversation

@YuriiMotov

@YuriiMotov YuriiMotov commented Feb 23, 2026

Copy link
Copy Markdown
Member

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

codspeed-hq Bot commented Feb 23, 2026

Copy link
Copy Markdown

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.

@tiangolo tiangolo 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.

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