Skip to content

chore(ci): add semantic pr scopes for test work#1571

Merged
webern merged 1 commit into
mainfrom
matt.briggs/scopes
May 4, 2026
Merged

chore(ci): add semantic pr scopes for test work#1571
webern merged 1 commit into
mainfrom
matt.briggs/scopes

Conversation

@webern
Copy link
Copy Markdown
Contributor

@webern webern commented May 4, 2026

Summary

I've had a few PRs where a scope that matches what I was doing didn't seem to exist. These were all while working on testing, so I have added a few scopes that seem to reasonably cover testing scopes.

Change Type

  • Non-functional (chore, refactoring, docs)

How did you test this PR?

N/A

References

N/A

@webern webern requested a review from a team as a code owner May 4, 2026 14:31
@dd-octo-sts dd-octo-sts Bot added the area/ci CI/CD, automated testing, etc. label May 4, 2026
Copy link
Copy Markdown
Member

@tobz tobz left a comment

Choose a reason for hiding this comment

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

We should actually add these to FIXED_SCOPES in tooling/update-pr-title-scopes.sh, and then run make update-pr-title-scopes to actually regenerate them.

CI will check if there's a mismatch after running make update-pr-title-scopes and fail otherwise.

@webern
Copy link
Copy Markdown
Contributor Author

webern commented May 4, 2026

We should actually add these to FIXED_SCOPES in tooling/update-pr-title-scopes.sh, and then run make update-pr-title-scopes to actually regenerate them.

CI will check if there's a mismatch after running make update-pr-title-scopes and fail otherwise.

Ah, sorry I didn't see that. Will do. Ok with the scopes?

I've had a few PRs where a scope that matches what I was doing didn't
seem to exist. These were all while working on testing, so I have added
a few scopes that seem to reasonably cover testing scopes.
@webern webern force-pushed the matt.briggs/scopes branch from dd5e70b to b6c3649 Compare May 4, 2026 14:58
@webern
Copy link
Copy Markdown
Contributor Author

webern commented May 4, 2026

Oh, it had actually drifted. Those unexpected changes came from make update-pr-title-scopes

Copy link
Copy Markdown
Member

@tobz tobz left a comment

Choose a reason for hiding this comment

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

Yeah, the fixed scopes being added here are fine by me. 👍🏻

@webern webern merged commit 8b9dadb into main May 4, 2026
68 of 69 checks passed
@webern webern deleted the matt.briggs/scopes branch May 4, 2026 15:37
dd-octo-sts Bot pushed a commit that referenced this pull request May 4, 2026
## Summary
I've had a few PRs where a scope that matches what I was doing didn't
seem to exist. These were all while working on testing, so I have added
a few scopes that seem to reasonably cover testing scopes.

## Change Type

- [x] Non-functional (chore, refactoring, docs)

## How did you test this PR?

N/A

## References

N/A 8b9dadb
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/ci CI/CD, automated testing, etc.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants