Skip to content

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

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:27
@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 fc3e919caaffe82fcd9a90e1f3b257594cb4bfab, the content this pull request carries at that commit against develop, and it recorded 0 findings.

@codecov

codecov Bot commented Nov 10, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 87.68%. Comparing base (f279ce2) to head (fc3e919).

Additional details and impacted files
@@           Coverage Diff            @@
##           develop     #486   +/-   ##
========================================
  Coverage    87.68%   87.68%           
========================================
  Files           11       11           
  Lines         1941     1941           
  Branches       252      252           
========================================
  Hits          1702     1702           
  Misses         166      166           
  Partials        73       73           
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟢 Approval recommended

No active review issues are identified, and the changes preserve existing reviewer settings while adding the intended filters for carried trees.

No open issues 🎉

What changed in this PR

Updates repository reviewer configurations to exclude carried, verbatim trees whose findings belong in the hub repository.

Changes:

  • Adds CodeRabbit path filters for .github/skills/** and lib/fleet/sh/**.
  • Adds matching Qodo ignore globs.
  • Preserves all existing reviewer settings.
File Description
.pr_agent.toml Adds Qodo ignore globs for the carried trees.
.coderabbit.yaml Adds CodeRabbit path filters for the carried trees.

Review effort: Lite

@ptr727
ptr727 merged commit 89b8791 into develop Nov 10, 2026
15 checks passed
@ptr727
ptr727 deleted the reviewer-skip-carried-trees branch November 10, 2026 00:33
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