catalyst / catalyst-moodle-workflows

4 stars 9 forks source link

Fix supported range checks for new ci #28

Closed keevan closed 2 years ago

keevan commented 2 years ago

Ensure for fixed supported ranges, the upper range will default to master if it doesn't exist in the matrix. This will mean something like $plugin->supported = [35, 500] will still work even if MOODLE_500_STABLE does not exist, and will just default that to master. If and when it does exist though, it should be added to the .github/actions/matrix/matrix_includes.yml file.

Currently this is including master when it shouldn't, as it is a closed range that is less than master. https://github.com/catalyst/moodle-tool_trigger/runs/6122019823?check_suite_focus=true