Skip to content

Commit aa1a049

Browse files
committed
updates
Signed-off-by: Jessie Frazelle <github@jessfraz.com>
1 parent 780f1d9 commit aa1a049

2 files changed

Lines changed: 8 additions & 7 deletions

File tree

.codex/AGENTS.md

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,6 +28,7 @@
2828
5. Implement the fix or ask about the tradeoffs the user is willing to make.
2929
- Write idiomatic, simple, maintainable code. Always ask yourself if this is the most simple intuitive solution to the problem.
3030
- Leave each repo better than how you found it. If something is giving a code smell, fix it for the next person.
31+
- Clean up unused code ruthlessly. If a function no longer needs a parameter or a helper is dead, delete it and update the callers instead of letting the junk linger.
3132
- **Search before pivoting**. If you are stuck or uncertain, do a quick web search for official docs or specs, then continue with the current approach. Do not change direction unless asked.
3233
- If code is very confusing or hard to understand:
3334
1. Try to simplify it.
@@ -48,7 +49,7 @@
4849
## Testing Philosophy
4950

5051
- I HATE MOCK tests, either do unit or e2e, nothing inbetween. Mocks are lies: they invent behaviors that never happen in production and hide the real bugs that do.
51-
- Test `EVERYTHING`. Our goal here is that any code we write, a new person coming into the repo cannot break, since there is a test that asserts the user facing behavior we want.
52+
- Test `EVERYTHING`. Tests must be rigorous. Our intent is ensuring a new person contributing to the same code base cannot break our stuff and that nothing slips by. We love rigour.
5253
- If tests live in the same Rust module as non-test code, keep them at the bottom inside `mod tests {}`; avoid inventing inline modules like `mod my_name_tests`.
5354
- Unless the user asks otherwise, run only the tests you added or modified instead of the entire suite to avoid wasting time.
5455

flake.lock

Lines changed: 6 additions & 6 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)