Skip to content

Skip the Carried Verbatim Trees in the Reviewer Configs - #246

Merged
ptr727 merged 1 commit into
developfrom
reviewer-skip-carried-trees
Nov 10, 2026
Merged

ptr727 merged 1 commit into
developfrom
reviewer-skip-carried-trees

Conversation

@ptr727

@ptr727 ptr727 commented Nov 10, 2026

Copy link
Copy Markdown
Owner

Summary

Writes the reviewer configs the hub now renders for this repository, from ptr727/ProjectTemplate's scripts/agent_policy.py at hub 9f3e03c (ptr727/ProjectTemplate#4020).

  • .coderabbit.yaml gains reviews.path_filters excluding .github/skills/** and lib/fleet/sh/**.
  • .pr_agent.toml gains an [ignore] glob naming the same trees.

Both trees are hub text the repository carries byte for byte, so a reviewer's finding on them belongs at the hub rather than here. Every other setting in both files is unchanged.

Verification

  • The diff is six added lines, three per file. Both files parse, and the existing settings and line endings are unchanged.
  • A recorded local strict review pass found nothing, and the diff-scoped prose gate is clean.
  • Qodo reads .pr_agent.toml from the default branch, so its exclusion binds once this reaches main.

Rendered from ptr727/ProjectTemplate's scripts/agent_policy.py at hub 9f3e03c.
CodeRabbit and Qodo now skip the trees the hub carries byte for byte, so a
finding on hub text lands at the hub rather than on this repository.
Copilot AI lite review requested due to automatic review settings November 10, 2026 00:28
@coderabbitai

coderabbitai Bot commented Nov 10, 2026

Copy link
Copy Markdown

Important

  • Trigger review

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

Run configuration

  • 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

@ptr727

ptr727 commented Nov 10, 2026

Copy link
Copy Markdown
Owner Author

A recorded local strict-review pass covers head e52d4cbdcd1fafc39eaa870ba8cec6a40774dbfe, the content this pull request carries at that commit against develop, and it recorded 0 findings.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 Approval recommended

The reviewer configurations consistently add the requested exclusions while preserving existing settings, leaving no unresolved review issues.

No open issues 🎉

What changed in this PR

Updates repository-level reviewer configurations so CodeRabbit and Qodo PR-Agent skip the carried .github/skills/** and lib/fleet/sh/** trees, whose findings belong in the hub repository.

Changes:

  • Adds the two exclusions to .coderabbit.yaml.
  • Adds matching ignore globs to .pr_agent.toml.
  • Preserves the existing reviewer settings.
  • The lib/fleet/sh/** exclusion currently matches no tree in the PR head and should be aligned with the repository’s carried content or omitted.
File Summary Review note
.pr_agent.toml Adds Qodo PR-Agent ignore globs for the carried trees. The lib/fleet/sh/** pattern currently matches no tree in the PR head.
.coderabbit.yaml Adds CodeRabbit path filters for the carried trees. The lib/fleet/sh/** pattern currently matches no tree in the PR head.

Review effort: Lite

@ptr727
ptr727 merged commit 0a41027 into develop Nov 10, 2026
13 checks passed
@ptr727
ptr727 deleted the reviewer-skip-carried-trees branch November 10, 2026 00:34
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.

2 participants