Skip to content

Commit d9cc0c9

Browse files
committed
add cargo check requirement to CLAUDE.md
1 parent c66cd52 commit d9cc0c9

File tree

1 file changed

+16
-6
lines changed

1 file changed

+16
-6
lines changed

CLAUDE.md

Lines changed: 16 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -34,12 +34,16 @@ THEGRAPH_STORE_POSTGRES_DIESEL_URL="postgresql://graph:graph@127.0.0.1:5432/grap
3434
# 🚨 MANDATORY: Format all code IMMEDIATELY after any .rs file edit
3535
cargo fmt --all
3636

37-
# Check code without building
37+
# 🚨 MANDATORY: Check code for warnings and errors - MUST have zero warnings
3838
cargo check
3939
```
4040

41-
🚨 **CRITICAL REMINDER**: `cargo fmt --all` is MANDATORY after editing ANY .rs file - NO EXCEPTIONS!
42-
This must be done from the project root BEFORE any commit. Forgetting this means you failed to follow instructions.
41+
🚨 **CRITICAL REQUIREMENTS for ANY implementation**:
42+
1. **`cargo fmt --all`** is MANDATORY after editing ANY .rs file - NO EXCEPTIONS!
43+
2. **`cargo check`** MUST show zero warnings before any commit - NO EXCEPTIONS!
44+
3. **All tests** MUST pass before any commit
45+
4. All requirements must be met from the project root BEFORE any commit
46+
5. Forgetting any of these means you failed to follow instructions
4347

4448
## High-Level Architecture
4549

@@ -91,12 +95,18 @@ Use format: `{crate-name}: {description}`
9195
- Keep commits logical and atomic
9296
- Use `git rebase -i` to clean up history before merging
9397

94-
### Testing Requirements
98+
### Implementation Success Criteria
99+
Before any commit or PR, ALL of the following MUST be satisfied:
100+
101+
1. **🚨 MANDATORY**: All tests MUST pass
102+
2. **🚨 MANDATORY**: `cargo check` MUST show zero warnings
103+
3. **🚨 MANDATORY**: `cargo fmt --all` MUST be run after editing ANY .rs file
104+
105+
**Testing Notes**:
95106
- Unit tests inline with source code
96-
- Integration tests require Docker Compose setup and additional environment dependencies
107+
- Integration tests require Docker Compose setup and additional environment dependencies
97108
- Claude cannot run integration tests due to missing environment dependencies
98109
- Claude must set `THEGRAPH_STORE_POSTGRES_DIESEL_URL` before running any tests
99-
- **🚨 MANDATORY**: Always run `cargo fmt --all` after editing .rs files, BEFORE committing
100110

101111
### Environment Variables
102112
- `GRAPH_LOG=debug`: Enable debug logging

0 commit comments

Comments
 (0)