CI: Quote regex to avoid syntax error

data-apis / array-api-compat

Compatibility layer for common array libraries to support the Array API

https://data-apis.org/array-api-compat/

MIT License

75 stars 25 forks source link

CI: Quote regex to avoid syntax error #58

Closed matthewfeickert closed 1 year ago

matthewfeickert commented 1 year ago

Regex on tag must be quoted to avoid YAML syntax error.

Amends PR #51

@asmeurer The CI is failing on merge to main as when you added the regex for the tags to PR #51 I didn't check it on my fork (when debugging I had removed the branch and tag block to run on push). The regex needs to be quoted, else a YAML syntax error occurs.

matthewfeickert commented 1 year ago

Evidence that this works and you won't have to keep dealing with slightly wrong CI:

$ git diff origin/ci/fix-regex 
diff --git a/.github/workflows/publish-package.yml b/.github/workflows/publish-package.yml
index 35f087d..91a276f 100644
--- a/.github/workflows/publish-package.yml
+++ b/.github/workflows/publish-package.yml
@@ -3,6 +3,7 @@ on:
   push:
     branches:
     - main
+    - debug/workflow
     tags:
     - '[0-9]+.[0-9]+'
     - '[0-9]+.[0-9]+.[0-9]+'