From a2a71f704243a964e04b1ea04aa36a727a87a373 Mon Sep 17 00:00:00 2001 From: Claude Date: Wed, 25 Mar 2026 02:25:56 +0000 Subject: [PATCH] docs: remove PDU_NO_FAIL_FAST recommendation from AI instructions The fail-fast default is better for iterative AI agent workflows where speed matters. The TEST_SKIP hints already handle environment-specific test failures, making the no-fail-fast recommendation unnecessary. The PDU_NO_FAIL_FAST option remains available in test.sh for CI and manual use. https://claude.ai/code/session_01Ptkb3AMQSEjH5P2MRYcF88 --- .github/copilot-instructions.md | 1 - AGENTS.md | 1 - CLAUDE.md | 1 - template/ai-instructions/shared.md | 1 - 4 files changed, 4 deletions(-) diff --git a/.github/copilot-instructions.md b/.github/copilot-instructions.md index a82456c3..45708a5b 100644 --- a/.github/copilot-instructions.md +++ b/.github/copilot-instructions.md @@ -18,4 +18,3 @@ Read and follow the CONTRIBUTING.md file in this repository for all code style c - Install toolchain before running tests: `rustup toolchain install "$(< rust-toolchain)" && rustup component add --toolchain "$(< rust-toolchain)" rustfmt clippy` - Run `FMT=true LINT=true BUILD=true TEST=true DOC=true ./test.sh` to validate changes. If a test fails with a hint about `TEST_SKIP`, follow the hint and rerun with the suggested variable. - **ALWAYS run the full test suite** (`FMT=true LINT=true BUILD=true TEST=true DOC=true ./test.sh`) before committing, regardless of how trivial the change seems — this includes documentation-only changes, comment edits, config changes, and refactors. The test suite checks formatting, linting, building, tests, and docs across multiple feature combinations; any type of change can break any of these checks. -- Set `PDU_NO_FAIL_FAST=true` to run all checks instead of stopping at the first failure — this lets you see which checks pass and which fail diff --git a/AGENTS.md b/AGENTS.md index a82456c3..45708a5b 100644 --- a/AGENTS.md +++ b/AGENTS.md @@ -18,4 +18,3 @@ Read and follow the CONTRIBUTING.md file in this repository for all code style c - Install toolchain before running tests: `rustup toolchain install "$(< rust-toolchain)" && rustup component add --toolchain "$(< rust-toolchain)" rustfmt clippy` - Run `FMT=true LINT=true BUILD=true TEST=true DOC=true ./test.sh` to validate changes. If a test fails with a hint about `TEST_SKIP`, follow the hint and rerun with the suggested variable. - **ALWAYS run the full test suite** (`FMT=true LINT=true BUILD=true TEST=true DOC=true ./test.sh`) before committing, regardless of how trivial the change seems — this includes documentation-only changes, comment edits, config changes, and refactors. The test suite checks formatting, linting, building, tests, and docs across multiple feature combinations; any type of change can break any of these checks. -- Set `PDU_NO_FAIL_FAST=true` to run all checks instead of stopping at the first failure — this lets you see which checks pass and which fail diff --git a/CLAUDE.md b/CLAUDE.md index f0fc2926..f887e81e 100644 --- a/CLAUDE.md +++ b/CLAUDE.md @@ -18,5 +18,4 @@ Read and follow the CONTRIBUTING.md file in this repository for all code style c - Install toolchain before running tests: `rustup toolchain install "$(< rust-toolchain)" && rustup component add --toolchain "$(< rust-toolchain)" rustfmt clippy` - Run `FMT=true LINT=true BUILD=true TEST=true DOC=true ./test.sh` to validate changes. If a test fails with a hint about `TEST_SKIP`, follow the hint and rerun with the suggested variable. - **ALWAYS run the full test suite** (`FMT=true LINT=true BUILD=true TEST=true DOC=true ./test.sh`) before committing, regardless of how trivial the change seems — this includes documentation-only changes, comment edits, config changes, and refactors. The test suite checks formatting, linting, building, tests, and docs across multiple feature combinations; any type of change can break any of these checks. -- Set `PDU_NO_FAIL_FAST=true` to run all checks instead of stopping at the first failure — this lets you see which checks pass and which fail - `gh` (GitHub CLI) is not installed — do not attempt to use it diff --git a/template/ai-instructions/shared.md b/template/ai-instructions/shared.md index a82456c3..45708a5b 100644 --- a/template/ai-instructions/shared.md +++ b/template/ai-instructions/shared.md @@ -18,4 +18,3 @@ Read and follow the CONTRIBUTING.md file in this repository for all code style c - Install toolchain before running tests: `rustup toolchain install "$(< rust-toolchain)" && rustup component add --toolchain "$(< rust-toolchain)" rustfmt clippy` - Run `FMT=true LINT=true BUILD=true TEST=true DOC=true ./test.sh` to validate changes. If a test fails with a hint about `TEST_SKIP`, follow the hint and rerun with the suggested variable. - **ALWAYS run the full test suite** (`FMT=true LINT=true BUILD=true TEST=true DOC=true ./test.sh`) before committing, regardless of how trivial the change seems — this includes documentation-only changes, comment edits, config changes, and refactors. The test suite checks formatting, linting, building, tests, and docs across multiple feature combinations; any type of change can break any of these checks. -- Set `PDU_NO_FAIL_FAST=true` to run all checks instead of stopping at the first failure — this lets you see which checks pass and which fail