do not check if matrix combination values in exclude exist in `incl…

…ude` when they contain `${{ }}` (fix #414)

rule_matrix.go

@@ -112,6 +112,14 @@ func filterMatchesMatrixRow(row []RawYAMLValue, filter RawYAMLValue) bool {
 	return false
 }
 
+func isYAMLStringWithExpression(val RawYAMLValue) bool {
+	s, ok := val.(*RawYAMLString)
+	if !ok {
+		return false
+	}
+	return ContainsExpression(s.Value)
+}
+
 func (rule *RuleMatrix) checkExclude(m *Matrix) {
 	if m.Exclude == nil || len(m.Exclude.Combinations) == 0 || (m.Include != nil && m.Include.ContainsExpression()) {
 		return
@@ -178,7 +186,7 @@ Outer:
 				continue
 			}
 
-			if filterMatchesMatrixRow(vs, a.Value) {
+			if isYAMLStringWithExpression(a.Value) || filterMatchesMatrixRow(vs, a.Value) {
 				continue
 			}
 

testdata/ok/issue-414_expression_in_matrix_exclude.yaml

@@ -0,0 +1,12 @@
+on: [push]
+
+jobs:
+  test:
+    runs-on: ubuntu-latest
+    strategy:
+      matrix:
+        foo: ['a', 'b']
+        exclude:
+          - foo: ${{ github.event_name }}
+    steps:
+      - run: echo ${{ matrix.foo }}