From 251039e3364108120cd2f6573e76f1d87139eac2 Mon Sep 17 00:00:00 2001 From: Claude Date: Wed, 18 Mar 2026 03:24:46 +0000 Subject: [PATCH] docs(ai): remove references to deleted CONTRIBUTING.md sections Remove `#[non_exhaustive]` from custom errors bullet and remove `#![deny(warnings)]` bullet, matching the removals in PR #358. https://claude.ai/code/session_012rU7VAeZADzFWewCEUMMyG --- .github/copilot-instructions.md | 3 +-- AGENTS.md | 3 +-- CLAUDE.md | 3 +-- 3 files changed, 3 insertions(+), 6 deletions(-) diff --git a/.github/copilot-instructions.md b/.github/copilot-instructions.md index 78756d14..ec6147da 100644 --- a/.github/copilot-instructions.md +++ b/.github/copilot-instructions.md @@ -11,9 +11,8 @@ Read and follow the CONTRIBUTING.md file in this repository for all code style c - Use descriptive variable and closure parameter names by default — single letters are only allowed in: conventional names (`n` for count, `f` for formatter), comparison closures (`|a, b|`), trivial single-expression closures, fold accumulators, index variables (`i`/`j`/`k` in short closures or index-based loops only), and test fixtures (identical roles only). Never use single letters in multi-line functions or closures - Prefer `where` clauses for multiple trait bounds - Derive order: std traits → comparison traits → `Hash` → derive_more → feature-gated -- Custom errors: `#[derive(Debug, Display, Error)]` + `#[non_exhaustive]` +- Custom errors: `#[derive(Debug, Display, Error)]` - Minimize `unwrap()` in non-test code — use proper error handling -- `#![deny(warnings)]` is active — code must be warning-free - Install toolchain before running tests: `rustup toolchain install "$(< rust-toolchain)" && rustup component add --toolchain "$(< rust-toolchain)" rustfmt clippy` - If the AI agent is Claude Code, `gh` (GitHub CLI) is not installed — do not attempt to use it - Run `FMT=true LINT=true BUILD=true TEST=true DOC=true ./test.sh` to validate changes diff --git a/AGENTS.md b/AGENTS.md index 78756d14..ec6147da 100644 --- a/AGENTS.md +++ b/AGENTS.md @@ -11,9 +11,8 @@ Read and follow the CONTRIBUTING.md file in this repository for all code style c - Use descriptive variable and closure parameter names by default — single letters are only allowed in: conventional names (`n` for count, `f` for formatter), comparison closures (`|a, b|`), trivial single-expression closures, fold accumulators, index variables (`i`/`j`/`k` in short closures or index-based loops only), and test fixtures (identical roles only). Never use single letters in multi-line functions or closures - Prefer `where` clauses for multiple trait bounds - Derive order: std traits → comparison traits → `Hash` → derive_more → feature-gated -- Custom errors: `#[derive(Debug, Display, Error)]` + `#[non_exhaustive]` +- Custom errors: `#[derive(Debug, Display, Error)]` - Minimize `unwrap()` in non-test code — use proper error handling -- `#![deny(warnings)]` is active — code must be warning-free - Install toolchain before running tests: `rustup toolchain install "$(< rust-toolchain)" && rustup component add --toolchain "$(< rust-toolchain)" rustfmt clippy` - If the AI agent is Claude Code, `gh` (GitHub CLI) is not installed — do not attempt to use it - Run `FMT=true LINT=true BUILD=true TEST=true DOC=true ./test.sh` to validate changes diff --git a/CLAUDE.md b/CLAUDE.md index 78756d14..ec6147da 100644 --- a/CLAUDE.md +++ b/CLAUDE.md @@ -11,9 +11,8 @@ Read and follow the CONTRIBUTING.md file in this repository for all code style c - Use descriptive variable and closure parameter names by default — single letters are only allowed in: conventional names (`n` for count, `f` for formatter), comparison closures (`|a, b|`), trivial single-expression closures, fold accumulators, index variables (`i`/`j`/`k` in short closures or index-based loops only), and test fixtures (identical roles only). Never use single letters in multi-line functions or closures - Prefer `where` clauses for multiple trait bounds - Derive order: std traits → comparison traits → `Hash` → derive_more → feature-gated -- Custom errors: `#[derive(Debug, Display, Error)]` + `#[non_exhaustive]` +- Custom errors: `#[derive(Debug, Display, Error)]` - Minimize `unwrap()` in non-test code — use proper error handling -- `#![deny(warnings)]` is active — code must be warning-free - Install toolchain before running tests: `rustup toolchain install "$(< rust-toolchain)" && rustup component add --toolchain "$(< rust-toolchain)" rustfmt clippy` - If the AI agent is Claude Code, `gh` (GitHub CLI) is not installed — do not attempt to use it - Run `FMT=true LINT=true BUILD=true TEST=true DOC=true ./test.sh` to validate changes