docs: add CONTRIBUTING.md guide for new contributors#484
Conversation
Provides a complete onboarding guide covering: - Quick start setup - Finding issues to work on (label guide) - Branch naming conventions - Development workflow (fork, branch, commit, PR) - Project structure overview with key file paths - Styling conventions and file size guidelines - i18n workflow for adding translations - Code principles (KISS, DRY, single responsibility) This gives new contributors everything they need in one place to start contributing effectively.
|
PR builds are available as GitHub Actions artifacts: https://github.com/NeuralNomadsAI/CodeNomad/actions/runs/26045596021 Artifacts expire in 7 days.
|
|
review it this CONTRIBUTING.md is always relevant and true, while keeping in mind the recent guide skills fro contributors #493 |
|
Reviewed Findings:
No code/doc changes were pushed from this review. Overall recommendation: request changes before merging, mainly to align the package map and navigation guidance with PR #493 and to fix the ambiguous styling paths/prerequisites. -- |
…isites - Add missing packages/opencode-plugin and packages/cloudflare to the package table. - Replace hardcoded Key UI Files table with a pointer to the codenomad-architecture-guide skill for maintainability. - Fix styling paths to use packages/ui/src/styles/... (repo-root relative) instead of src/styles/... - Add Prerequisites section (Node.js 18+, OpenCode CLI in PATH) before Quick Start. - Broaden typecheck guidance in PR checklist to recommend root or area-specific scripts.
The reviewer recommended keeping the file list available as a supplement until PR NeuralNomadsAI#493 (codenomad-architecture-guide skill) is merged, since the skill file does not exist in this branch yet.
|
PR builds are available as GitHub Actions artifacts: https://github.com/NeuralNomadsAI/CodeNomad/actions/runs/26466827670 Artifacts expire in 7 days.
|
|
@CodeNomadBot |
|
Re-ran the same The previous concerns are mostly addressed:
Remaining finding:
Recommendation: fix the table separator before merging. After that, I do not see remaining accuracy issues from the previous review scope. -- |
CodeNomadBot flagged that the 2-column Package/Description table had a 3-cell separator (|---|---|---|), which renders inconsistently. Fixed to |---|---| to match the number of columns.
Summary
Adds a CONTRIBUTING.md file to help new contributors get started.
Contents
This gives new contributors everything they need in one place to start contributing effectively.