Skip to content

Add a PR auto-labeler workflow - #6799

Merged
pietern merged 2 commits into
mainfrom
add-pr-auto-labeler
Sep 22, 2026
Merged

pietern merged 2 commits into
mainfrom
add-pr-auto-labeler

Conversation

@pietern

@pietern pietern commented Sep 22, 2026

Copy link
Copy Markdown
Contributor

Adds a GitHub Actions workflow (actions/labeler v5) that labels PRs from the paths they touch, driven by a glob→label map in .github/labeler.yml. New label mappings are added by editing that file.

Runs on pull_request_target so it can label fork PRs, with minimal permissions (contents: read, pull-requests: write) and scoped to main. It never checks out or runs PR code, and sync-labels is off so it only adds labels, never removing manually-applied ones.

This pull request and its description were written by Isaac.

Adds a GitHub Actions workflow (actions/labeler v5) that labels PRs from the paths they touch, driven by a glob->label map in .github/labeler.yml. New label mappings are added by editing that file.

Runs on pull_request_target so it can label fork PRs, with minimal permissions (contents: read, pull-requests: write) and scoped to main. It never checks out or runs PR code, and sync-labels is off so it only adds labels, never removing manually-applied ones.

Co-authored-by: Isaac <no-reply@databricks.com>
@eng-dev-ecosystem-bot

eng-dev-ecosystem-bot commented Sep 22, 2026 •

Copy link
Copy Markdown
Collaborator

Integration test report

Commit: da44c30

Run: 35739078433

Env 🔄​flaky ✅​pass 🙈​skip Time
✅​ aws linux 276 15 5:59
✅​ aws windows 278 13 5:00
✅​ azure linux 275 15 5:43
✅​ azure windows 277 13 5:06
✅​ gcp linux 276 15 5:54
🔄​ gcp windows 1 277 13 6:03
Test Name gcp windows
🔄​ TestAccept 🔄​f
Top 3 slowest tests (at least 2 minutes):
duration env testname
4:57 aws windows TestAccept
3:14 azure windows TestAccept
2:38 gcp windows TestAccept

Co-authored-by: Isaac <no-reply@databricks.com>
@pietern
pietern requested a review from simonfaltum September 22, 2026 14:20
@pietern
pietern enabled auto-merge September 22, 2026 14:25
Comment on lines +33 to +34
# Only add labels; never remove labels that were applied manually or by other automation.
sync-labels: false

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.

why not remove them if the PR changes scope? e.g. accidentally touches DABs, gets labelled, refactored, keeps DABs

Comment thread .github/labeler.yml
Comment on lines +32 to +34
- 'bundle/**'
- 'cmd/bundle/**'
- 'acceptance/bundle/**'

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.

nit: in case of changelog-only changes?

Suggested change
- 'bundle/**'
- 'cmd/bundle/**'
- 'acceptance/bundle/**'
- '.nextchanges/bundles/*'
- 'bundle/**'
- 'cmd/bundle/**'
- 'acceptance/bundle/**'

@pietern
pietern added this pull request to the merge queue Sep 22, 2026
Merged via the queue into main with commit 870ab36 Sep 22, 2026
39 checks passed
@pietern
pietern deleted the add-pr-auto-labeler branch September 22, 2026 14:54
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.

5 participants