Refresh contributor docs, add PR template, code of conduct, and code standards - #2575
Refresh contributor docs, add PR template, code of conduct, and code standards#2575lancepioch wants to merge 1 commit into
Conversation
Fixes the broken Windows-style vendor/bin paths in contributing.md (now composer pint / composer phpstan), refreshes the dev environment setup (Herd-first, frontend assets, verified artisan commands), documents the pre-PR checks and the CLA Assistant flow, and links the user docs. Adds a pull request template, a Contributor Covenant 2.1 code of conduct (contact: team@pelican.dev), and code-standards.md covering Pint, Larastan, Pest, the API contract, and codebase conventions; cross-links the code of conduct from contributing.md and security.md.
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (5)
Included review availability: Your plan provides up to 8 included reviews per hour; 5 remain after this review. 📝 WalkthroughWalkthroughThe changes add contribution standards, setup and validation guidance, a pull request checklist, a Contributor Covenant code of conduct, and a link between vulnerability reporting and the code of conduct. ChangesContribution Documentation
Priority: ⬇️ Low Merge Risk: ⚪ Minimal · up to This documentation-only change has no remaining material merge risk. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
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 |
Fixes the broken
\vendor\binpaths in contributing.md (nowcomposer pint/composer phpstan), refreshes the dev environment setup (Herd-first with SQLite, frontend asset build, verified artisan commands), documents the pre-PR checks and the CLA Assistant signing flow, and links the user docs.Adds
.github/PULL_REQUEST_TEMPLATE.md, a Contributor Covenant 2.1code_of_conduct.md(contact: team@pelican.dev, cross-linked from contributing.md and security.md), andcode-standards.mdcovering Pint, Larastan (including the forbiddenapp()/resolve()rule), Pest suites, the API contract freeze, and codebase conventions.The supported-versions wording in security.md is left to #2567; labels and good-first-issue seeding are handled outside this PR. The CLA is already enforced by
cla.yaml, so no workflow changes.