Skip to content

ci: auto-label DataFusion issues and PRs - #3278

Merged
CTTY merged 1 commit into
mainfrom
kevinjqliu-add-datafusion-labeler
Sep 28, 2026
Merged

CTTY merged 1 commit into
mainfrom
kevinjqliu-add-datafusion-labeler

Conversation

@kevinjqliu

@kevinjqliu kevinjqliu commented Sep 24, 2026 •

Copy link
Copy Markdown
Contributor

Which issue does this PR close?

  • N/A

What changes are included in this PR?

Goal: automatically add the datafusion label to DataFusion-related issues and PRs. Only committers and collaborators can add labels, so these usually go unlabeled.

How: a small workflow using github/issue-labeler that adds datafusion when the title or body mentions datafusion (case-insensitive).

  • Runs on issues and PRs when opened or edited
  • Uses pull_request_target so fork PRs get labeled too. It never checks out PR code, and the regex config is read from main
  • Only adds labels, never removes them
  • github/* actions are on the ASF allowlist by default

Why not actions/labeler (used by iceberg and iceberg-go)?

  • It only labels PRs based on changed files (or branch names)
  • It can't read the title/body and doesn't handle issues
  • The DataFusion code no longer lives in this repo, so there are no files to match on

Are these changes tested?

Ran actionlint and zizmor locally. Workflows on these triggers only run from main, so we'll see it in action after merge.

AI Disclosure

Written with help from GitHub Copilot.

Contributors without triage permission cannot add labels. Use
github/issue-labeler to apply the datafusion label when an issue or PR
title or body mentions DataFusion.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Copilot AI lite review requested due to automatic review settings September 24, 2026 22:38

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 review overview

🟢 Approval recommended

No unresolved issues were identified in the reviewed changes.

Review effort: Lite
Findings: None

What changed in this PR

Adds automatic datafusion labeling for matching issues and pull requests.

Changes:

  • Adds issue and pull request labeling workflow.
  • Configures case-insensitive title/body matching.
  • Uses pinned github/issue-labeler without checking out untrusted code.
File Description
.github/​workflows/​labeler.yml Runs the labeler workflow with appropriate permissions.
.github/​labeler.yml Defines the DataFusion matching rule.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@kevinjqliu

Copy link
Copy Markdown
Contributor Author

This is reusable, I just want to start with the new datafusion label

@comphead comphead left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thanks @kevinjqliu looks reasonable for me

@CTTY CTTY left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM!

@CTTY
CTTY added this pull request to the merge queue Sep 28, 2026
Merged via the queue into main with commit 86d6618 Sep 28, 2026
25 checks passed
@CTTY
CTTY deleted the kevinjqliu-add-datafusion-labeler branch September 28, 2026 18:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants