From 43130aa40ed08ca937f700822a23c7e429817892 Mon Sep 17 00:00:00 2001 From: Doxa Maps Date: Mon, 28 Sep 2026 11:21:29 +0000 Subject: [PATCH] CONTRIBUTING: source and build output as two commits in one push (not two pushes), with the two exemptions --- documentation/CONTRIBUTING.md | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/documentation/CONTRIBUTING.md b/documentation/CONTRIBUTING.md index f42d5198d..7818461df 100644 --- a/documentation/CONTRIBUTING.md +++ b/documentation/CONTRIBUTING.md @@ -53,6 +53,12 @@ it (script tag or the generated per-map page). Same-origin vs external-CDN modes - Never bake tokens or data into a bundle — everything arrives at runtime via `profile-config`. - Before a PR: `bun run build && bun run verify` must pass (adds drop-portability + parse gates). - Colors are values, not logic: [documentation/contributing/COLORS.md](./contributing/COLORS.md). +- A landing that changes source AND its build output arrives as **two commits in one push**: the source + change first, then a `rebuild ` commit carrying only `public/`. Reviewers read the first; the + second is machine output. Don't split them across two pushes: the landing check rebuilds every affected + bundle on the commit about to reach main and refuses one whose `public/` bundle no longer matches its + source. Exempt: a `public/`-only rebuild after someone else's source landed, and a comment-only source + change (comments are stripped from the build, so there is no `public/` half). ## One-line recap