Add contributing guide - #290
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (2)
🚧 Files skipped from review as they are similar to previous changes (2)
📝 WalkthroughWalkthroughAdded ChangesContribution documentation
Estimated code review effort: 1 (Trivial) | ~5 minutes 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
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.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@CONTRIBUTING.md`:
- Line 3: Update the introductory description in CONTRIBUTING.md to use the
hyphenated compound modifier “open-source” in the phrase describing FreeFlow,
leaving the surrounding wording unchanged.
🪄 Autofix
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro Plus
Run ID: ed705860-179c-429e-9381-b95ff9e0850a
📒 Files selected for processing (3)
CHANGELOG.mdCONTRIBUTING.mdREADME.md
Introduce CONTRIBUTING.md documenting how to fork, build, test, and submit PRs to FreeFlow. Covers prerequisites (macOS, Xcode tools), build/test commands (make, make test), branch and commit conventions, changelog format, PR description expectations, and issue/discussion guidance. Link it from the README and add a changelog entry.
aa96423 to
68b8d56
Compare
Summary
Introduces
CONTRIBUTING.md— a guide documenting how to set up the project, build, test, and submit changes to FreeFlow. Also links it from the README and adds a changelog entry.Motivation
FreeFlow has no documented contribution process today — no
CONTRIBUTINGING.md, no issue/PR templates, no contributor guide. New contributors have to reverse-engineer the process from merged PRs and the Makefile. This lowers the barrier for community participation.The guide reflects the conventions already in use across the 108 merged PRs: fork-and-PR workflow,
make/make testfor building, focused PRs, changelog entries under[Unreleased], and responsive engagement with the CodeRabbit review bot. It does not invent new rules.Behavior
No code or runtime behavior changes. Adds a documentation file, a README link, and a changelog entry.
Changes
CONTRIBUTING.md(new) — Covers prerequisites (macOS 13+, Xcode CLI tools, Groq key), getting the source, building (make,make run,make ARCH=universal), testing (make test, how to extend the test runner), development conventions (branch naming, commit messages, code style, changelog format), submitting changes (PR description structure, size guidance, CodeRabbit review), reporting issues, and discussions.README.md(+4) — Adds a## Contributingsection linking toCONTRIBUTING.md.CHANGELOG.md(+6) — Adds an[Unreleased]→Addedentry for the guide.Test plan
CONTRIBUTING.mdcontent reflects actual repo conventions (verified against Makefile targets, existing PR patterns, CHANGELOG format, and test runner structure).make/make testunaffected.Notes
If the maintainers prefer a different structure or scope (e.g., shorter, split into separate files, or covering additional topics like release process or security reporting), happy to adjust.
Summary by CodeRabbit