Skip to content

Conversation

nathan-weinberg
Copy link
Member

Auto-labeler hasn't been updated in a bit, making some additions here with existing files/labels for maintainer ease

Signed-off-by: Nathan Weinberg <nweinber@redhat.com>
@nathan-weinberg nathan-weinberg changed the title [ci] Update labeler.yml with additional files/labels ci: Update labeler.yml with additional files/labels Apr 24, 2024
Comment on lines 23 to +33
- any-glob-to-any-file:
- CODE_OF_CONDUCT.md
- CONTRIBUTOR_RULES.md
- governance.md
- MAINTAINERS.md
- README.md
- SECURITY.md
- TROUBLESHOOTING.md
- CONTRIBUTING/**/*
- docs/**/*
- MAINTAINERS/**/*
Copy link
Member

Choose a reason for hiding this comment

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

It looks like this supports globbing, so *.md may be better for the top level?

Copy link
Member Author

Choose a reason for hiding this comment

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

Thought about that, but some Markdown files aren't docs necessarily, e.g. PULL_REQUEST_TEMPLATE.md

CI/CD:
- changed-file:
- any-glob-to-any-file:
- .github/labeler.yml
Copy link
Member

Choose a reason for hiding this comment

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

How about .github/*

Copy link
Member Author

Choose a reason for hiding this comment

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

PULL_REQUEST_TEMPLATE.md shouldn't be included IMO

@nathan-weinberg nathan-weinberg added the CI/CD Affects CI/CD configuration label Apr 24, 2024
@nathan-weinberg nathan-weinberg merged commit dc95f0b into instructlab:main Apr 24, 2024
@nathan-weinberg nathan-weinberg deleted the addn-labels branch April 24, 2024 23:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI/CD Affects CI/CD configuration
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants