Skip to content

[CI] Skip multimodal CI for doc-only changes#21334

Merged
hnyls2002 merged 7 commits intomainfrom
fix-multimodal
Mar 24, 2026
Merged

[CI] Skip multimodal CI for doc-only changes#21334
hnyls2002 merged 7 commits intomainfrom
fix-multimodal

Conversation

@hnyls2002
Copy link
Copy Markdown
Collaborator

@hnyls2002 hnyls2002 commented Mar 24, 2026

Summary

Test plan

Exclude .md and .ipynb files under python/sglang/multimodal_gen/
from triggering multimodal CI tests via dorny/paths-filter.
@gemini-code-assist
Copy link
Copy Markdown
Contributor

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

dorny/paths-filter uses OR logic, so negation patterns like
!*.md match almost everything and defeat the purpose. Switch to
explicitly listing code file extensions (.py, .cu, .cpp, etc.)
to correctly skip doc-only changes.
Replace explicit extension whitelist with picomatch extglob negation
`*.!(md|ipynb)` so any new code file type is automatically included.
@github-actions github-actions bot added documentation Improvements or additions to documentation diffusion SGLang Diffusion labels Mar 24, 2026
@hnyls2002 hnyls2002 merged commit 31c35f1 into main Mar 24, 2026
22 of 33 checks passed
@hnyls2002 hnyls2002 deleted the fix-multimodal branch March 24, 2026 21:07
0-693 pushed a commit to 0-693/sglang that referenced this pull request Mar 25, 2026
johnnycxm pushed a commit to johnnycxm/sglang that referenced this pull request Mar 25, 2026
johnnycxm pushed a commit to johnnycxm/sglang that referenced this pull request Mar 25, 2026
@OrangeRedeng OrangeRedeng mentioned this pull request Mar 26, 2026
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

diffusion SGLang Diffusion documentation Improvements or additions to documentation run-ci

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant