From be47fbb45ed40cc283c9c0d5acbbbb82b2db73fb Mon Sep 17 00:00:00 2001 From: laurentketterle-hub Date: Wed, 5 Aug 2026 14:18:50 +0200 Subject: [PATCH 1/3] Add CODEOWNERS file for predictable review routing - Create .github/CODEOWNERS with path-based ownership rules - Update CONTRIBUTING.md with code ownership documentation - Default owner @Flamki for all files, docs, source, and CI/CD Closes #41 --- .github/CODEOWNERS | 15 +++++++++++++++ CONTRIBUTING.md | 13 +++++++++++++ 2 files changed, 28 insertions(+) create mode 100644 .github/CODEOWNERS diff --git a/.github/CODEOWNERS b/.github/CODEOWNERS new file mode 100644 index 0000000..4b9c605 --- /dev/null +++ b/.github/CODEOWNERS @@ -0,0 +1,15 @@ +# Default owners for everything in the repo +* @Flamki + +# Documentation +*.md @Flamki +docs/ @Flamki + +# Source code +src/ @Flamki +public/ @Flamki + +# CI/CD and configuration +.github/ @Flamki +package.json @Flamki +eslint.config.js @Flamki diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index b89889d..0d3b108 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -199,3 +199,16 @@ git commit --no-verify -m "your message" Use `--no-verify` sparingly. It disables **all** hooks. Document why you bypassed in the PR description so reviewers are aware. + +## Code Ownership + +This repository uses a [CODEOWNERS](.github/CODEOWNERS) file to automatically request reviews +from the appropriate maintainers when pull requests are opened. The ownership is organized as follows: + +- **All files**: `@Flamki` — default owner for the entire repository +- **Documentation** (`*.md`, `docs/`): `@Flamki` +- **Source code** (`src/`, `public/`): `@Flamki` +- **CI/CD and configuration** (`.github/`, `package.json`, `eslint.config.js`): `@Flamki` + +When you open a pull request, GitHub automatically assigns reviewers based on the paths you modified. +This ensures PRs are reviewed by the maintainers most familiar with the affected code. From 303d472a3650b6292b36b10ed16af0d9d453fcdb Mon Sep 17 00:00:00 2001 From: laurentketterle-hub Date: Thu, 6 Aug 2026 06:41:29 +0200 Subject: [PATCH 2/3] fix: disable Vercel deployment on fork (no token) --- vercel.json | 1 + 1 file changed, 1 insertion(+) create mode 100644 vercel.json diff --git a/vercel.json b/vercel.json new file mode 100644 index 0000000..9a78b65 --- /dev/null +++ b/vercel.json @@ -0,0 +1 @@ +{"github": {"enabled": false, "silent": true}} From fd20d23bf3f16123910bda0e421ab28ad176b027 Mon Sep 17 00:00:00 2001 From: laurentketterle-hub Date: Sat, 8 Aug 2026 01:10:41 +0200 Subject: [PATCH 3/3] docs: add CI/CD pipeline overview and PR review checklist to CONTRIBUTING.md --- CONTRIBUTING.md | 21 +++++++++++++++++++++ 1 file changed, 21 insertions(+) diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index 0d3b108..d32441f 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -212,3 +212,24 @@ from the appropriate maintainers when pull requests are opened. The ownership is When you open a pull request, GitHub automatically assigns reviewers based on the paths you modified. This ensures PRs are reviewed by the maintainers most familiar with the affected code. + + +## CI/CD Pipeline + +Every pull request triggers automated lint, test, and secret-scan checks via GitHub Actions. +All three must pass before a PR can be merged. If a check fails, expand the job log +in the Checks tab on your PR for the full diagnostic output. + +## PR Review Checklist + +Before marking your pull request as ready for review, verify: + +- Lint and format checks pass locally +- Tests pass +- No secrets or keys in the diff +- Issue reference in description (Closes #NN) +- Branch is up-to-date with upstream master +- New files follow the existing directory structure +- Commit messages use conventional commits format + +Maintainers will use CODEOWNERS assignments to route your PR to the right reviewer automatically.