Skip to content

feat: add GitHub Actions workflows for automatic labeling of PRs#3134

Closed
AaronGoldsmith wants to merge 1 commit intoblock:mainfrom
AaronGoldsmith:ag/dev-x-github-labeler
Closed

feat: add GitHub Actions workflows for automatic labeling of PRs#3134
AaronGoldsmith wants to merge 1 commit intoblock:mainfrom
AaronGoldsmith:ag/dev-x-github-labeler

Conversation

@AaronGoldsmith
Copy link
Contributor

Github supports organizing issues/pull requests using labels.

This pull request introduces automated labeling for pull requests based on file changes, using GitHub Actions. It includes configuration for label rules, workflows to add labels when a pull request is opened, and workflows to synchronize labels when a pull request is updated or reopened.

Configuration for automated labeling:

  • .github/labeler.yml: Added rules to assign labels based on file paths and patterns, covering areas like documentation, CLI, Rust backend, UI/Desktop application, Windows-specific changes, and dependency updates.

Workflows for applying and synchronizing labels:

Comment on lines +55 to +58
# Demonstration of labeler for initial Pull Request commit
labeler:
- changed-files:
- any-glob-to-any-file: '.github/labeler.yml'
Copy link
Contributor Author

Choose a reason for hiding this comment

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

The purpose of adding this was so reviewers could better understand how the labeler works, but it looks like it needs approvals from maintainers to run. https://github.com/block/goose/actions/runs/15937664559

Let me know if I should remove this bit.

@AaronGoldsmith AaronGoldsmith marked this pull request as ready for review June 27, 2025 23:18
@jamadeo jamadeo requested a review from DOsinga July 3, 2025 14:02
@jamadeo
Copy link
Collaborator

jamadeo commented Jul 3, 2025

@DOsinga can we use this with your proposed labeling?

@DOsinga
Copy link
Collaborator

DOsinga commented Jul 3, 2025

pretty neat! my labelling proposal was mostly meant for issues, not for PRs but still. maybe we can get goose at some point to do the issue labelling...

Comment on lines +29 to +32
- '**/*.tsx'
- '**/*.ts'
- '**/*.js'
- '**/*.jsx'
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I wanted to call out that the Goose docs are written in typescript, which might not fit under "ui". I'll let others provide their opinions

@DOsinga
Copy link
Collaborator

DOsinga commented Jul 3, 2025

actually I think the docs are in markdown - there's some ts plumbing though

@DOsinga DOsinga self-assigned this Jul 16, 2025
@DOsinga
Copy link
Collaborator

DOsinga commented Jul 29, 2025

closing due to inactivity. feel free to reopen

@DOsinga DOsinga closed this Jul 29, 2025
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.

3 participants