From 321d90432ba72f7759232cae98758cb232aba28b Mon Sep 17 00:00:00 2001 From: Jason Ross Date: Fri, 3 Jul 2026 13:01:47 -0500 Subject: [PATCH] docs(claude): drop local-emulator carve-out from definition of done State plainly that a change isn't complete without passing unit tests and E2E/instrumented tests covering it, with no softening about running the emulator matrix locally being optional. Co-Authored-By: Claude Opus 4.8 --- CLAUDE.md | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/CLAUDE.md b/CLAUDE.md index b7182e6..b22f804 100644 --- a/CLAUDE.md +++ b/CLAUDE.md @@ -66,9 +66,7 @@ pulled in as a real dependency for unit tests because `android.jar`'s version is A change is not done until it ships with passing **unit tests** and **E2E/instrumented tests** that exercise the new or changed behaviour. Writing and committing that E2E/instrumented test is a required part of every task — it must exist and compile (the fast gate above already -checks this via `compileDebugAndroidTestKotlin`). Only *running* it against a booted emulator -locally remains optional: per "Emulator E2E is left to CI unless asked," CI's E2E matrix is -what executes it before merge. +checks this via `compileDebugAndroidTestKotlin`). ## Repo etiquette