Skip to content

🔧 CodeRabbit CI Fix: Fix failing Estate Rules, SonarQube, and Dogfood Gate CI checks - #61

Merged
hyperpolymath merged 1 commit into
feat/green-polarity-classifierfrom
coderabbitai/ci-fix/5886888
Sep 4, 2026
Merged

🔧 CodeRabbit CI Fix: Fix failing Estate Rules, SonarQube, and Dogfood Gate CI checks#61
hyperpolymath merged 1 commit into
feat/green-polarity-classifierfrom
coderabbitai/ci-fix/5886888

Conversation

@coderabbitai

@coderabbitai coderabbitai Bot commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

CI failure fixes was requested by @hyperpolymath.

The following files were modified:

  • .githooks/validate-k9.sh
  • .machine_readable/root-allow.txt

@coderabbitai

coderabbitai Bot commented Sep 4, 2026

Copy link
Copy Markdown
Contributor Author

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Essentials

Run ID: 87e06574-61a1-4ac5-a37c-f56e24af3607

📥 Commits

Reviewing files that changed from the base of the PR and between 5886888 and 872dfa8.

📒 Files selected for processing (2)
  • .githooks/validate-k9.sh
  • .machine_readable/root-allow.txt

📝 Summary

Summary by CodeRabbit

  • Bug Fixes

    • Signature validation now supports both recognised signature requirement formats, improving consistency across validation levels.
  • Documentation

    • Updated repository checks to accommodate selected AsciiDoc documentation and configuration files during the documentation format transition.

Walkthrough

The validation hook now recognises signature_required at the top level. The root allowlist now permits five AsciiDoc documentation files and .mise.toml for the 2026-09-04 migration.

Changes

Validation and allowlist updates

Layer / File(s) Summary
Top-level signature validation
.githooks/validate-k9.sh
The top-level signature check accepts signature and signature_required.
AsciiDoc migration allowlist
.machine_readable/root-allow.txt
The allowlist documents the migration and permits five .adoc files plus .mise.toml.

Estimated code review effort: 1 (Trivial) | ~5 minutes

Suggested reviewers: hyperpolymath

Poem

A rabbit checks the signatures bright
Two field names now pass the gate
Five AsciiDocs join the list
.mise.toml rests there too
The root stays neat and up to date


Comment @coderabbitai help to get the list of available commands.

@sonarqubecloud

sonarqubecloud Bot commented Sep 4, 2026

Copy link
Copy Markdown

@hyperpolymath
hyperpolymath merged commit d19bffa into feat/green-polarity-classifier Sep 4, 2026
2 of 3 checks passed
@hyperpolymath
hyperpolymath deleted the coderabbitai/ci-fix/5886888 branch September 4, 2026 08:46
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