Skip to content

Conversation

@iiio2
Copy link
Contributor

@iiio2 iiio2 commented Jan 31, 2026

Recently I've learned about skills. Suppose we're using skills. Then we may create those 2 folders. maybe we need to put that in gitignore.

@iiio2 iiio2 requested a review from pi0 as a code owner January 31, 2026 13:30
@vercel
Copy link

vercel bot commented Jan 31, 2026

@iiio2 is attempting to deploy a commit to the Nitro Team on Vercel.

A member of the Team first needs to authorize it.

@coderabbitai
Copy link

coderabbitai bot commented Jan 31, 2026

📝 Walkthrough

Walkthrough

The change adds new ignore patterns to the .gitignore file for AI configuration directories (".agents" and ".claude") with an organizational section header.

Changes

Cohort / File(s) Summary
Gitignore Configuration
.gitignore
Added "# AI config" section header with two new ignore entries for ".agents" and ".claude" directories.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title 'chore: add ai config in gitignore' follows conventional commits format with appropriate 'chore' type and clearly describes the main change.
Description check ✅ Passed The description is related to the changeset, explaining the rationale for adding AI config folders to gitignore when using skills.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@pkg-pr-new
Copy link

pkg-pr-new bot commented Jan 31, 2026

Open in StackBlitz

npm i https://pkg.pr.new/nitrojs/nitro@3993

commit: 34cff22

@pi0 pi0 merged commit eea737f into nitrojs:main Jan 31, 2026
10 of 11 checks passed
@iiio2
Copy link
Contributor Author

iiio2 commented Jan 31, 2026

Thanks. 😊

@iiio2 iiio2 deleted the chore/add-ai-config branch January 31, 2026 14:48
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.

2 participants