Skip to content

Conversation

JaredforReal
Copy link
Contributor

What type of PR is this?
feat: Update .gitignore for AI docs

What this PR does / why we need it:
Avoid pushing AI docs(cursor, claude, copilot) to the remote repo mistakenly.

Signed-off-by: JaredforReal <w13431838023@gmail.com>
@JaredforReal JaredforReal marked this pull request as ready for review September 22, 2025 02:43
Copy link

netlify bot commented Sep 22, 2025

Deploy Preview for vllm-semantic-router ready!

Name Link
🔨 Latest commit 6b413ac
🔍 Latest deploy log https://app.netlify.com/projects/vllm-semantic-router/deploys/68d0b7e6db4101000850a7b2
😎 Deploy Preview https://deploy-preview-191--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.

Copy link

👥 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:

  • .gitignore

vLLM

🎉 Thanks for your contributions!

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

Copy link
Contributor

@tao12345666333 tao12345666333 left a comment

Choose a reason for hiding this comment

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

I don't understand the value of this PR. Can you explain it in detail?

@JaredforReal
Copy link
Contributor Author

JaredforReal commented Sep 22, 2025

@tao12345666333 I mistakenly pushed my copilot-instruction.md to this repo this morning, and rootfs suggested adding gitignore for this kind of AI doc to avoid the similar mistake. orz

@rootfs rootfs merged commit b7888be into vllm-project:main Sep 22, 2025
9 checks passed
@JaredforReal JaredforReal deleted the ignore branch September 24, 2025 16:36
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