Skip to content

Commit 57cfb84

Browse files
author
Dennis Braun
committed
CONTRIBUTING: require issue before features, one PR per change
1 parent 0bece51 commit 57cfb84

File tree

1 file changed

+15
-3
lines changed

1 file changed

+15
-3
lines changed

CONTRIBUTING.md

Lines changed: 15 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -70,13 +70,25 @@ Translations live in `app/i18n/` as JSON files:
7070

7171
Each file has a `_meta` field with `language_name` and `flag`. When adding or changing UI strings, update **all 4 files**.
7272

73-
## Guidelines
73+
## Before You Start
7474

75-
- Keep changes focused and minimal
75+
**Please open an issue first** before working on any new feature or significant change. This lets us discuss the approach and make sure it fits the project direction. PRs without a prior issue may be closed.
76+
77+
This is especially important for:
78+
- New features or modules
79+
- Architectural changes
80+
- Changes touching multiple files
81+
82+
Small bugfixes and typo corrections are fine without an issue.
83+
84+
## Pull Request Guidelines
85+
86+
- **One PR per feature/fix.** Don't bundle unrelated changes.
87+
- **Keep changes focused and minimal.** Smaller PRs are easier to review and more likely to be merged.
7688
- Add tests for new functionality
7789
- Maintain all 4 language translations (EN/DE/FR/ES) in `app/i18n/*.json`
78-
- CHANGELOG entries must be in English
7990
- Run the full test suite before submitting a PR
91+
- AI-generated bulk PRs without prior discussion will not be merged
8092

8193
## Adding Modem Support
8294

0 commit comments

Comments
 (0)