Skip to content

Conversation

@hl475
Copy link
Collaborator

@hl475 hl475 commented Oct 20, 2025

Summary

Previously, we skip if every changed file is docs/**, this PR enhances the logic to skip pipeline if every changed file is docs/** OR **/*.md OR mkdocs.yaml

Test

Test on PR with changes on both docs and others

vllm-project/vllm#25116
=>
https://buildkite.com/vllm/ci/builds/35549/steps/canvas

Test on PR with changes on docs only

vllm-project/vllm#25367
=>
https://buildkite.com/vllm/ci/builds/35550/steps/canvas

Test on PR with no changes on docs

vllm-project/vllm#27113
=>
https://buildkite.com/vllm/ci/builds/35547/steps/canvas

Test on merged PR with changes on both docs and others

vllm-project/vllm@f574383
=>
https://buildkite.com/vllm/ci/builds/35546/steps/canvas

Test on merged PR with changes on docs only

vllm-project/vllm@e6e898f
=>
https://buildkite.com/vllm/ci/builds/35545/steps/canvas

Test on merged PR with no changes on docs

vllm-project/vllm@87efc68
=>
https://buildkite.com/vllm/ci/builds/35544/steps/canvas

hl475 added 2 commits October 19, 2025 23:03
Signed-off-by: Huamin Li <[email protected]>
Signed-off-by: Huamin Li <[email protected]>
@hl475 hl475 changed the title [WIP] enhance_doc_only Enhance the doc only change detection logic to skip the pipeline Oct 20, 2025
@hl475 hl475 requested a review from khluu October 20, 2025 18:46
Copy link
Collaborator

@khluu khluu left a comment

Choose a reason for hiding this comment

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

Thank you!

@khluu khluu merged commit a64d73f into main Oct 24, 2025
2 checks passed
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.

3 participants