Skip to content

fix(ci): replace invalid mergify review team with owner user - #504

Merged
KooshaPari merged 1 commit into
mainfrom
fix/mergify-main-team-fix
Aug 7, 2026
Merged

KooshaPari merged 1 commit into
mainfrom
fix/mergify-main-team-fix

Conversation

@KooshaPari

@KooshaPari KooshaPari commented Aug 5, 2026 •

Copy link
Copy Markdown
Owner

Replace invalid Mergify request_reviews team target and align rule with existing ownership behavior.

Changes:

  • Change .mergify.yml on main from:
    • request_reviews.teams: [phenotype/core]
    • to request_reviews.users: [KooshaPari]
  • Keep governance self-check guard: author != KooshaPari so owner PRs do not trigger self-review requests.

Why:

Copilot AI lite review requested due to automatic review settings August 5, 2026 23:32
@chatgpt-codex-connector

Copy link
Copy Markdown

Codex usage limits have been reached for code reviews. Please check with the admins of this repo to increase the limits by adding credits.
Repo admins can enable using credits for code reviews in their settings.

@codeant-ai

codeant-ai Bot commented Aug 5, 2026 •

Copy link
Copy Markdown

🤖 CodeAnt AI — Review Status

Status Commit Started (UTC) Finished (UTC)
✅ Reviewed your PR 96a9c0c Aug 05, 2026 · 23:32 23:33

@codeant-ai

codeant-ai Bot commented Aug 5, 2026

Copy link
Copy Markdown

Thanks for using CodeAnt! 🎉

We're free for open-source projects. if you're enjoying it, help us grow by sharing.

Share on X ·
Reddit ·
LinkedIn

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.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@codeant-ai codeant-ai Bot added the size:XS This PR changes 0-9 lines, ignoring generated files label Aug 5, 2026
@coderabbitai

coderabbitai Bot commented Aug 5, 2026 •

Copy link
Copy Markdown

Review Change Stack

Note

.coderabbit.yaml has unrecognized properties

CodeRabbit is using all valid settings from your configuration. Unrecognized properties (listed below) have been ignored and may indicate typos or deprecated fields that can be removed.

⚠️ Parsing warnings (1)
Validation error: Unrecognized key: "review"
⚙️ Configuration instructions
  • Please see the configuration documentation for more information.
  • You can also validate your configuration using the online YAML validator.
  • If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: # yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json
📝 Walkthrough

Walkthrough

The Mergify review-request rule now excludes PRs authored by KooshaPari, removes the phenotype/core team target, and retains the configured user reviewer.

Changes

Review request automation

Layer / File(s) Summary
Reviewer targeting rule
.mergify.yml
The rule excludes KooshaPari as an author and requests review from the configured user instead of the phenotype/core team.

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

Possibly related PRs

Suggested reviewers: copilot

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The description explains the configuration change and reason but omits most template sections, including validation, tests, coverage, reviewer notes, and the 71-pillar self-check. Add the required template sections and record validation results, test status, coverage notes, reviewer notes, and the applicable 71-pillar self-check.
✅ Passed checks (4 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Title check ✅ Passed The title clearly identifies the Mergify configuration fix and replacement of an invalid review team with the owner user.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/mergify-main-team-fix

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

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

@sonarqubecloud

sonarqubecloud Bot commented Aug 5, 2026

Copy link
Copy Markdown

@github-actions

github-actions Bot commented Aug 5, 2026

Copy link
Copy Markdown

L17 Latency Budget Report

--- Latency Budget Summary ---
  Total endpoints checked: 0
  Passed: 0
  Warnings: 0
  Failures: 0

Checked against: budgets/rest-endpoints.yaml.

@github-actions

github-actions Bot commented Aug 5, 2026

Copy link
Copy Markdown

L17 Latency Regression Report

No trace file available — cannot compute regression

Threshold: 10% p99 regression.

@github-actions

github-actions Bot commented Aug 5, 2026

Copy link
Copy Markdown

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

None

@kilo-code-bot

kilo-code-bot Bot commented Aug 6, 2026 •

Copy link
Copy Markdown

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Files Reviewed (1 file)
  • .mergify.yml

Reviewed by ling-3.0-flash:free · Input: 91.6K · Output: 5.6K · Cached: 192.1K

@KooshaPari
KooshaPari merged commit 0de07de into main Aug 7, 2026
24 of 28 checks passed
@KooshaPari
KooshaPari deleted the fix/mergify-main-team-fix branch August 7, 2026 06:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:XS This PR changes 0-9 lines, ignoring generated files

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants