Skip to content

Labeler: Add labeler glob patterns for CI#16915

Closed
Ayesh wants to merge 1 commit intophp:masterfrom
Ayesh:labeler-cat-ci
Closed

Labeler: Add labeler glob patterns for CI#16915
Ayesh wants to merge 1 commit intophp:masterfrom
Ayesh:labeler-cat-ci

Conversation

@Ayesh
Copy link
Copy Markdown
Member

@Ayesh Ayesh commented Nov 24, 2024

We have a "Category: CI" label in GH issues and PRs, but there is no labeler rule to automatically mark PRs as such.

This adds a labeler rule to automatically add the "Category: CI" label if the PR changes any file in:
- .circleci/*
- .github/*

If all the changes are within these files, it will not be labelled.
- .github/ISSUE_TEMPLATE/*
- .github/lsan-suppressions.txt

@Ayesh Ayesh requested a review from TimWolla as a code owner November 24, 2024 14:24
- any-glob-to-any-file:
- .circleci/*
- .github/actions/**/*
- .github/CODEOWNERS
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.

If we consider this to be CI, we might as well consider anything in .github/ to be CI. The only exceptions I'm seeing right now are lsan-suppressions.txt and labeler.yml itself.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I was trying to exclude ISSUE_TEMPLATE folder and potentially future files like funding, code of conduct, etc files that are merely templates.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Hi @TimWolla I rebased this PR, and hoping to revive it. Could I request another review please? I changed the glob patterns to include .github/* directory entirely, with two exceptions for ISSUE_TEMPLATE and lsan-suppressions.txt.

@Girgias
Copy link
Copy Markdown
Member

Girgias commented Apr 6, 2026

Is this PR still being purused?

We have a "Category: CI" label in GH issues and PRs, but there is no
labeler rule to automatically mark PRs as such.

This adds a labeler rule to automatically add the "Category: CI" label
if the PR changes _any_ file in:

  - '.circleci/*'
  - '.github/*'

If all the changes are within these files, it will _not_ be labelled.

 - '.github/ISSUE_TEMPLATE/*'
 - '.github/lsan-suppressions.txt'
@Ayesh Ayesh requested a review from TimWolla April 6, 2026 15:51
@Ayesh
Copy link
Copy Markdown
Member Author

Ayesh commented Apr 6, 2026

@Girgias I had forgotten about it 🥲. We still do not have a Category: CI labeling pattern, so I rebased and updated the PR, with a simpler glob pattern.

I could use your review if possible, thank you.

@Girgias
Copy link
Copy Markdown
Member

Girgias commented Apr 6, 2026

@Girgias I had forgotten about it 🥲. We still do not have a Category: CI labeling pattern, so I rebased and updated the PR, with a simpler glob pattern.

I could use your review if possible, thank you.

Not usually what I deal with, maybe @iluuu1994 is more suited for a review tho :)

@Ayesh Ayesh closed this in c445f1b Apr 6, 2026
@iluuu1994
Copy link
Copy Markdown
Member

Was just about to merge, thanks! Small note: The indentation of the rest of the doc is weird (last level only has 2 spaces). Would be nice to fix that.

Ayesh added a commit that referenced this pull request Apr 6, 2026
Fixes mixed indentation from the previous PR GH-16915.
@Ayesh
Copy link
Copy Markdown
Member Author

Ayesh commented Apr 6, 2026

Thank you @iluuu1994, I'm sorry I just realized after merging 🥲.

I also just pushed a fix, fixing the mixed indentation.

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.

4 participants