Skip to content

infra: add in PR labeler workflow#1229

Open
AbeCoull wants to merge 4 commits into
mainfrom
labeler
Open

infra: add in PR labeler workflow#1229
AbeCoull wants to merge 4 commits into
mainfrom
labeler

Conversation

@AbeCoull
Copy link
Copy Markdown
Contributor

Issue #, if available:

Description of changes:

Adds in automatic labeling for PRs. For coloring the Labels or adding description, this will be done after the code is merged.

Testing done:

Used in a personal repo already. Example: AbeCoull/prism-q#3

Merge Checklist

Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your pull request.

General

Tests

  • I have added tests that prove my fix is effective or that my feature works (if appropriate)
  • I have checked that my tests are not configured for a specific region or account (if appropriate)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@AbeCoull AbeCoull requested review from rmshaffer and speller26 March 11, 2026 16:12
@AbeCoull AbeCoull requested a review from a team as a code owner March 11, 2026 16:12
Comment thread .github/labeler.yml Outdated
- any-glob-to-any-file:
- 'src/braket/program_sets/**'

tests:
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

This may be a bit overdone to include. I can keep or delete depending on where other opinions fall

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.

I think it's fine, it may be noisy but there may be some test-only PRs which would be nice to have a label for.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I am going to omit tests until I can find a good way to have that be a standalone label. That is to say if there are other labels, test won't appear.

@AbeCoull AbeCoull changed the title ci: add in PR labeler workflow infra: add in PR labeler workflow Mar 11, 2026
@codecov
Copy link
Copy Markdown

codecov Bot commented Mar 11, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (50346a9) to head (21c3fc0).

Additional details and impacted files
@@            Coverage Diff            @@
##              main     #1229   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files          169       169           
  Lines        10963     10963           
  Branches      1412      1412           
=========================================
  Hits         10963     10963           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Comment thread .github/labeler.yml Outdated
- any-glob-to-any-file:
- 'src/braket/program_sets/**'

tests:
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.

I think it's fine, it may be noisy but there may be some test-only PRs which would be nice to have a label for.

steps:
- uses: actions/labeler@v5
with:
sync-labels: true
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.

I agree with this

Comment thread .github/labeler.yml Outdated
Comment on lines +43 to +46
annealing:
- changed-files:
- any-glob-to-any-file:
- 'src/braket/annealing/**'
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.

curious about how you chose which folders to include labels for vs. not? Some folders (e.g. parametric, registers, tasks, tracking, timings, experimental_capabilities) are not included in this list. Is it worth having a label for every folder, or should we perhaps have an "other source" label for folders we don't expect to change much?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I made some consolidations. The main aim isn't, at least in the first pass, to capture every folder. It is aimed at having some discoverability and look into the type of changes going in from the PR. The next commit aims to better capture that.

Comment thread .github/labeler.yml Outdated
Comment thread .github/labeler.yml
Comment thread .github/labeler.yml Outdated
@AbeCoull AbeCoull requested a review from rmshaffer May 14, 2026 18:31
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.

2 participants