Skip to content

Commit ec8df2c

Browse files
tsenartampcode-com
andcommitted
docs: add pre-push checklist to AGENTS.md
Amp-Thread-ID: https://ampcode.com/threads/T-019c66f5-f25d-701a-9ea4-deb56277f474 Co-authored-by: Amp <amp@ampcode.com>
1 parent 3c282fd commit ec8df2c

1 file changed

Lines changed: 14 additions & 2 deletions

File tree

AGENTS.md

Lines changed: 14 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -73,11 +73,23 @@ CLI (clap)
7373
```sh
7474
cargo build # Debug build
7575
cargo build --release # Release (LTO=fat, codegen-units=1)
76-
cargo test --all # Run all 90 tests
76+
cargo test --all # Run all tests
7777
cargo fmt --check # Verify formatting
78-
cargo clippy -- -D warnings # Lint (CI runs this, must pass)
78+
cargo clippy -- -D warnings # Lint CI treats warnings as errors
7979
```
8080

81+
### Pre-push checklist
82+
83+
**Run all three before every push.** CI will reject the PR/push if any fail.
84+
85+
```sh
86+
cargo fmt
87+
cargo clippy -- -D warnings
88+
cargo test --all
89+
```
90+
91+
This is non-negotiable. Don't push and hope CI passes.
92+
8193
## CI & Release
8294

8395
CI runs on every push/PR via `.github/workflows/ci.yml`. Three jobs:

0 commit comments

Comments
 (0)