-
-
Notifications
You must be signed in to change notification settings - Fork 1k
Move vouch requirement to top of CONTRIBUTING.md #3029
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Move vouch requirement to top of CONTRIBUTING.md #3029
Conversation
Contributors need to be vouched before opening PRs, but this requirement was buried far down in the document. This change: - Adds mention of vouches in the intro paragraph - Moves the "Getting vouched" section to right after the intro This makes the requirement more visible to new contributors. Slack thread: https://triggerdotdev.slack.com/archives/C0A7Q6F62NS/p1770805895370749 https://claude.ai/code/session_01G6VVbgfUAeCpJfedELdqq1
|
WalkthroughThis pull request reorganizes the CONTRIBUTING.md documentation to prioritize the vouching requirement. The changes consolidate duplicate "Getting vouched" guidance by removing a later instance and introducing a new section near the top that outlines the vouching requirement and a four-step process for obtaining vouching. The developing/setup section is expanded with more explicit details about the release workflow and PR target branch (main). Minor wording adjustments reflect these structural changes. Net change is +14/-14 lines. Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes 🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 inconclusive)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches🧪 Generate unit tests (beta)
No actionable comments were generated in the recent review. 🎉 📜 Recent review detailsConfiguration used: Repository UI Review profile: CHILL Plan: Pro 📒 Files selected for processing (1)
🧰 Additional context used📓 Path-based instructions (1)**/*.{js,ts,jsx,tsx,json,md,yaml,yml}📄 CodeRabbit inference engine (AGENTS.md)
Files:
🧠 Learnings (4)📚 Learning: 2026-01-15T10:48:02.687ZApplied to files:
📚 Learning: 2025-11-27T16:27:35.304ZApplied to files:
📚 Learning: 2025-11-27T16:27:35.304ZApplied to files:
📚 Learning: 2025-11-27T16:26:37.432ZApplied to files:
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (22)
🔇 Additional comments (2)
✏️ Tip: You can disable this entire section by setting 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. Comment |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Contributors need to be vouched before opening PRs, but this requirement was buried far down in the document. This change:
This makes the requirement more visible to new contributors.
Slack thread: https://triggerdotdev.slack.com/archives/C0A7Q6F62NS/p1770805895370749 https://claude.ai/code/session_01G6VVbgfUAeCpJfedELdqq1