catalyst / catalyst-moodle-workflows

5 stars 10 forks source link

issue #116: use fixed fetch-depth value #118

Closed djarran closed 3 months ago

djarran commented 3 months ago

Description: The previous fix in #117 for Issue #116 did not work - the ternary was still evaluated incorrectly. Instead of being evaluated to 2, it was evaluating to true:

(success() && contains(github.event_name, 'push') && (steps.check-branch.outputs.publishable == 'true'))

Instead of using a ternary, we now use a fixed number (20) for fetch-depth.