Skip to content

Conversation

@RocMarshal
Copy link
Contributor

What changes were proposed in this pull request

Issue Number: close #4319

Brief change log

Add boring-cyborg.yml for pull-request infra

Verifying this change

N.A

Does this pull request potentially affect one of the following parts

  • Dependencies (does it add or upgrade a dependency): (yes / no)

@github-actions github-actions bot added the INFRA label Dec 30, 2025
@RocMarshal RocMarshal marked this pull request as ready for review January 12, 2026 08:42
@RocMarshal RocMarshal requested a review from Copilot January 17, 2026 15:22
@xinzhuxiansheng xinzhuxiansheng self-requested a review January 17, 2026 15:22
@RocMarshal RocMarshal requested review from wolfboys and removed request for xinzhuxiansheng January 17, 2026 15:22
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This pull request adds a boring-cyborg.yml configuration file to automate repository infrastructure tasks such as PR labeling, title validation, issue linking, and first-time contributor messages.

Changes:

  • Added boring-cyborg.yml configuration file with PR labeling rules, title validation, issue link insertion, and welcome messages for new contributors

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines +35 to +38
- streampark-console/streampark-console-service/**

component=streampark-console-webapp:
- streampark-console/streampark-console-webapp/**
Copy link

Copilot AI Jan 17, 2026

Choose a reason for hiding this comment

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

The path pattern is inconsistent with other entries. Lines 35 and 38 use /** (two asterisks) without the trailing /*, while all other component paths use /**/* (two asterisks followed by slash and one asterisk). For consistency, these should also use /**/* pattern.

Suggested change
- streampark-console/streampark-console-service/**
component=streampark-console-webapp:
- streampark-console/streampark-console-webapp/**
- streampark-console/streampark-console-service/**/*
component=streampark-console-webapp:
- streampark-console/streampark-console-webapp/**/*

Copilot uses AI. Check for mistakes.
Comment on lines +63 to +66
titleIssueIdRegexp: \[(ISSUE-[0-9]+)\]
# the issue link to be added. ${1}, ${2} ... are replaced with the match groups from the
# title match (remember to use quotes)
descriptionIssueLink: "[${1}](https://github.com/apache/streampark/issues/${1})"
Copy link

Copilot AI Jan 17, 2026

Choose a reason for hiding this comment

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

The issue link URL may be incorrect. The pattern captures "ISSUE-[0-9]+" (e.g., "ISSUE-4319") and uses it directly in the GitHub issues URL. However, GitHub issue URLs typically use just the numeric ID (e.g., "/issues/4319" not "/issues/ISSUE-4319"). The captured group should extract only the numeric part, or the URL pattern should strip the "ISSUE-" prefix.

Suggested change
titleIssueIdRegexp: \[(ISSUE-[0-9]+)\]
# the issue link to be added. ${1}, ${2} ... are replaced with the match groups from the
# title match (remember to use quotes)
descriptionIssueLink: "[${1}](https://github.com/apache/streampark/issues/${1})"
titleIssueIdRegexp: \[ISSUE-([0-9]+)\]
# the issue link to be added. ${1}, ${2} ... are replaced with the match groups from the
# title match (remember to use quotes)
descriptionIssueLink: "[ISSUE-${1}](https://github.com/apache/streampark/issues/${1})"

Copilot uses AI. Check for mistakes.
@RocMarshal RocMarshal requested review from xinzhuxiansheng and removed request for wolfboys January 17, 2026 15:28
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copy link
Contributor

@xinzhuxiansheng xinzhuxiansheng left a comment

Choose a reason for hiding this comment

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

LGTM

@RocMarshal RocMarshal closed this Jan 17, 2026
@RocMarshal RocMarshal reopened this Jan 17, 2026
@sonarqubecloud
Copy link

@RocMarshal RocMarshal merged commit 252bd20 into apache:dev Jan 17, 2026
66 of 82 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Umbrella] Add boring-cyborg.yml for repository infrastructure

2 participants