Skip to content

[Workflow] Add PR labeler workflow#1460

Open
Steven Ramirez Rosa (Steven6798) wants to merge 1 commit into
qualcomm:mainfrom
Steven6798:issue-1457
Open

[Workflow] Add PR labeler workflow#1460
Steven Ramirez Rosa (Steven6798) wants to merge 1 commit into
qualcomm:mainfrom
Steven6798:issue-1457

Conversation

@Steven6798

Copy link
Copy Markdown
Contributor

Add workflow to automatically label PRs. This will be used for automatically generating release notes.

Fix: #1457

@Steven6798

Copy link
Copy Markdown
Contributor Author

If anybody has ideas for more labels, please let me know.

@Steven6798 Steven Ramirez Rosa (Steven6798) changed the title [Workflow] Add PR labler workflow [Workflow] Add PR labeler workflow Jul 13, 2026

@parth-07 Parth (parth-07) left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Do we need to add labels to PRs? Isn't it enough to add labels to the issue?

What happens if a PR modifies all target files? Will we then include that fix/PR for all targets?

Comment thread .github/workflows/new-pr.yml Outdated
Comment thread .github/pr-labeler.yml
@Steven6798

Steven Ramirez Rosa (Steven6798) commented Jul 14, 2026

Copy link
Copy Markdown
Contributor Author

Do we need to add labels to PRs? Isn't it enough to add labels to the issue?

The workflow that generates release notes use labels of PRs

What happens if a PR modifies all target files? Will we then include that fix/PR for all targets?

This workflow just assigns some basic labels to help. Label assignment is not 100% automated. The owner of the PR should check if the labels are appropriate for the PR and add or remove as needed. This is just a helper.

@Steven6798

Copy link
Copy Markdown
Contributor Author

This is not exclusive to release notes. We can add labels here to help us track changes to certain pieces of ELD.

Add workflow to automatically label PRs. This will be used
for automatically generating release notes.

Fix: qualcomm#1457

Signed-off-by: Steven Ramirez Rosa <ramirezr@qti.qualcomm.com>

@quic-areg quic-areg left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

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.

[Workflow] Add PR labler workflow

3 participants