ci: fix commit type prefix regexp in release script (#8046)

mafredri · johnstcn · web-flow · commit 3f81f38fcc19 · 2023-06-15T10:57:18.000Z
Previously we were incorrectly categorizing `fix(a/b): ...` as other
because the regexp only expected letters, not `/`. Now we accept any
input within the parenthesis.

---------

Co-authored-by: Cian Johnston &lt;cian@coder.com&gt;
diff --git a/scripts/release/check_commit_metadata.sh b/scripts/release/check_commit_metadata.sh
@@ -40,12 +40,12 @@ ignore_missing_metadata=${CODER_IGNORE_MISSING_COMMIT_METADATA:-0}
 
 main() {
 	# Match a commit prefix pattern, e.g. feat: or feat(site):.
-	prefix_pattern="^([a-z]+)(\([a-z]*\))?:"
+	prefix_pattern="^([a-z]+)(\([^)]+\))?:"
 
 	# If a commit contains this title prefix or the source PR contains the
 	# label, patch releases will not be allowed.
 	# This regex matches both `feat!:` and `feat(site)!:`.
-	breaking_title="^[a-z]+(\([a-z]*\))?!:"
+	breaking_title="^[a-z]+(\([^)]+\))?!:"
 	breaking_label=release/breaking
 	breaking_category=breaking
 	experimental_label=release/experimental
