Skip to content

OWNERS: Auto Sync OWNERS files from community membership - #24542

Merged
ti-chi-bot[bot] merged 2 commits into
masterfrom
bot/update-owners-1792726898391
Oct 27, 2026
Merged

ti-chi-bot[bot] merged 2 commits into
masterfrom
bot/update-owners-1792726898391

Conversation

@ti-chi-bot

@ti-chi-bot ti-chi-bot commented Oct 23, 2026 •

Copy link
Copy Markdown
Member

Check List

Tests

  • No need to test
    • I checked and no code files have been changed.

Summary by CodeRabbit

  • Chores
    • Updated the community approver roster: four entries were removed, while the two remaining approvers are unchanged. No application features or functionality changed.

[skip ci] Update OWNERS file

skip-checks: true
Signed-off-by: Ti Chi Robot <ti-community-prow-bot@tidb.io>
@ti-chi-bot ti-chi-bot Bot added missing-translation-status This PR does not have translation status info. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Oct 23, 2026
@coderabbitai

coderabbitai Bot commented Oct 23, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

No actionable comments were generated in the recent review.

Recent review info
Run configuration
Commits

Reviewing files that changed from the base of the PR and between 34a945b and 0f61eb5.

Files selected for processing: 1
  • OWNERS_ALIASES
Files with no reviewable changes: 1
  • OWNERS_ALIASES

Included review availability: This review used your included allowance. Your plan provides up to 4 included reviews per hour; 1 remain after this review.

Walkthrough

The change removes Icemap, Oreoxmt, hfxsd, and jackysp from sig-community-approvers. breezewish and csuzhangxc remain.

Changes: Community SIG ownership aliases

Layer / File(s) Summary
Update approver membership
OWNERS_ALIASES
sig-community-approvers no longer includes Icemap, Oreoxmt, hfxsd, or jackysp. breezewish and csuzhangxc remain.

Priority: Low

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

Change: Other

Suggested reviewers: lilin90

Merge Risk: Minimal · up to 0f61e

No actionable merge-blocking risk is identified; the change appears mergeable after normal checks.

Pre-merge checks: Passed 4 · Failed 1

Failed Failed checks: 1 warning

Check name Status Explanation Resolution
Description check Warning The description does not follow the template. It keeps only the 'Check List' section with 'No need to test' ticked, and the reason given is that no code files changed. The template sections for what c… Add the template sections, and at minimum fill in: (1) 'What is changed, added, or deleted?' stating that the OWNERS_ALIASES entries for Icemap, Oreoxmt, hfxsd, and jackysp are removed from sig-community-approvers, and why. (2) 'Which TiDB …
Passed Passed checks: 4 passed
Check name Status Explanation
Title check Passed The title names the change: OWNERS files are auto-synced from community membership. The raw summary confirms the change removes four users from the sig-community-approvers alias in OWNERS_ALIASES. The…
Docstring Coverage Passed No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Documentation-only edits and deletions within survi…
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.
Full details: Title check

Explanation

The title names the change: OWNERS files are auto-synced from community membership. The raw summary confirms the change removes four users from the sig-community-approvers alias in OWNERS_ALIASES. The title is specific and accurate, though it does not name the removed approvers. It is 56 characters, which is within a reasonable length.

Full details: Docstring Coverage

Explanation

No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Documentation-only edits and deletions within surviving functions are included; unchanged diff context and fully deleted functions are excluded. Analyzed 0 functions across 0 files.

Full details: Description check

Explanation

The description does not follow the template. It keeps only the 'Check List' section with 'No need to test' ticked, and the reason given is that no code files changed. The template sections for what changed, the affected TiDB versions, related PR links, and AI agent involvement are missing. The required 'What is changed, added, or deleted?' and 'Which TiDB version(s) do your changes apply to?' sections are absent, so the description is largely incomplete.

Resolution

Add the template sections, and at minimum fill in: (1) 'What is changed, added, or deleted?' stating that the OWNERS_ALIASES entries for Icemap, Oreoxmt, hfxsd, and jackysp are removed from sig-community-approvers, and why. (2) 'Which TiDB version(s) do your changes apply to?' with the applicable boxes ticked, most likely master only. (3) The related PR link (PR #24542) under 'What is the related PR or file link(s)?'. Also complete the first-time contributors' checklist if applicable, and tick 'Change aliases' under 'Do your changes match any of the following descriptions?' since this PR changes aliases.

Finishing Touches
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • 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

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

@ti-chi-bot

Copy link
Copy Markdown
Member Author

/release-note-none

@ti-chi-bot ti-chi-bot added approved lgtm skip-issue-check Indicates that a PR no need to check linked issue. translation/no-need No need to translate this PR. labels Oct 23, 2026
@ti-chi-bot ti-chi-bot Bot removed the missing-translation-status This PR does not have translation status info. label Oct 23, 2026
@lilin90 lilin90 self-assigned this Oct 27, 2026

@lilin90 lilin90 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

/approve

@ti-chi-bot

ti-chi-bot Bot commented Oct 27, 2026

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: lilin90

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Comment thread OWNERS_ALIASES Outdated
@ti-chi-bot
ti-chi-bot Bot merged commit fa984e3 into master Oct 27, 2026
12 checks passed
@lilin90
lilin90 deleted the bot/update-owners-1792726898391 branch October 27, 2026 07:45
@lilin90 lilin90 added the ONCALL Relates to documentation oncall. label Oct 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved lgtm ONCALL Relates to documentation oncall. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. skip-issue-check Indicates that a PR no need to check linked issue. translation/no-need No need to translate this PR.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants