Skip to content

docs: describe the new test override behavior of go-ftw - #312

Open
theseion wants to merge 1 commit into
coreruleset:mainfrom
theseion:document-go-ftw-overrides
Open

theseion wants to merge 1 commit into
coreruleset:mainfrom
theseion:document-go-ftw-overrides

Conversation

@theseion

Copy link
Copy Markdown
Member

Updates the testing guide for the new test override behavior of go-ftw, introduced in coreruleset/go-ftw#693 (issue coreruleset/go-ftw#692).

  • forcepass/forcefail replace ignore in the example; overridden tests are run and the actual and forced result are reported.
  • Documents stale overrides (fail by default, stale_overrides: warn to only report).
  • Notes that ignore is deprecated (treated like forcepass) and that exclude is no longer supported in the configuration file (use --exclude for a single run and platform overrides for permanent platform differences).

Do not merge before the go-ftw change has been released, the guide describes behavior that older versions do not have.

🤖 Generated with Claude Code

Tests with `forcepass`/`forcefail` overrides are now run and reported
with their actual and forced result, stale overrides fail the run by
default, `ignore` is deprecated and `exclude` is no longer supported in
the configuration file.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Nov 12, 2026

Copy link
Copy Markdown

Important

  • Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

Run configuration

  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: Central YAML (base), Organization UI (inherited)
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 48076557-1b72-4d87-b8be-2a33820bf91d
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

Share

@theseion
theseion requested review from M4tteoP and fzipi November 12, 2026 10:05

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant