Skip to content

Conversation

@rootfs
Copy link
Collaborator

@rootfs rootfs commented Nov 7, 2025

Reverts #609

@yossiovadia this breaks the CI, can you run it locally and add the changed files too?

@rootfs rootfs requested a review from Xunzhuo as a code owner November 7, 2025 17:18
@netlify
Copy link

netlify bot commented Nov 7, 2025

Deploy Preview for vllm-semantic-router ready!

Name Link
🔨 Latest commit 01aa9d0
🔍 Latest deploy log https://app.netlify.com/projects/vllm-semantic-router/deploys/690e29eb4e238400085763fe
😎 Deploy Preview https://deploy-preview-610--vllm-semantic-router.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@github-actions
Copy link

github-actions bot commented Nov 7, 2025

👥 vLLM Semantic Team Notification

The following members have been identified for the changed files in this PR and have been automatically assigned:

📁 Root Directory

Owners: @rootfs, @Xunzhuo
Files changed:

  • .pre-commit-config.yaml

vLLM

🎉 Thanks for your contributions!

This comment was automatically generated based on the OWNER files in the repository.

@rootfs rootfs requested a review from yossiovadia November 7, 2025 17:18
@rootfs rootfs merged commit 025e69a into main Nov 7, 2025
6 of 8 checks passed
@rootfs rootfs deleted the revert-609-fix/pre-commit-yaml-lint-hook branch November 7, 2025 17:22
@yossiovadia
Copy link
Collaborator

HI @rootfs , we have chicken and egg :)
see #593 comment
if u prefer we can have seperate PR for the lint issue ( not part of @senan ) , let me know

@rootfs
Copy link
Collaborator Author

rootfs commented Nov 7, 2025

@yossiovadia @senan please make a separate lint pr, thanks!

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.

4 participants