Skip to content

Commit

Permalink
Move ignore exclusion if label up in the matrix definition
Browse files Browse the repository at this point in the history
Increase the readability of the matrix configurations by specifying
`ignore exclusion if` label between `config` and `suite` labels because
within the `suite` label there are made references to both
`config` and `ignore exclusion if` labels.
  • Loading branch information
findinpath committed Oct 20, 2022
1 parent f97c4a9 commit 7f4eda6
Showing 1 changed file with 11 additions and 11 deletions.
22 changes: 11 additions & 11 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -672,6 +672,17 @@ jobs:
- default
- hdp3
# TODO: config-apache-hive3
ignore exclusion if:
# Do not use this property outside of the matrix configuration.
#
# This is added to all matrix entries so they may be conditionally
# excluded by adding them to the excludes list with a GHA expression
# for this property.
# - If the expression evaluates to true, it will never match the a
# actual value of the property, and will therefore not be excluded.
# - If the expression evaluates to false, it will match the actual
# value of the property, and the exclusion will apply normally.
- "false"
suite:
- suite-1
- suite-2
Expand Down Expand Up @@ -722,17 +733,6 @@ jobs:
ignore exclusion if: >-
${{ secrets.DATABRICKS_TOKEN != '' }}
ignore exclusion if:
# Do not use this property outside of the matrix configuration.
#
# This is added to all matrix entries so they may be conditionally
# excluded by adding them to the excludes list with a GHA expression
# for this property.
# - If the expression evaluates to true, it will never match the a
# actual value of the property, and will therefore not be excluded.
# - If the expression evaluates to false, it will match the actual
# value of the property, and the exclusion will apply normally.
- "false"
include:
# this suite is not meant to be run with different configs
- config: default
Expand Down

0 comments on commit 7f4eda6

Please sign in to comment.