ci: enforce usage of scopes in PR title #3672
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Purpose of change
followup of #3536, enforces much stricter PR title convention for
from now on, changelog guidelines will be strictly enforced via semantic PR bot.
it means adding invalid mod ids like
fix(mods/foobar): baz
to PR title is now forbidden.Describe the solution
scripts/semantic.ts
that grabs all mainlined mod ids to autogenerate appropriatemods/<MOD_ID>
scope. this allows future tool to add tag/sort/group PRs by affected mods. Currently it's meant to be ran manually since we don't add/remove mainlined mods often.Alternatives
maybe
map
scopes could be useful.Additional Context
please mention if the script is confusing or too complex. I'm eager to improve documentation.