Concurrency, dynamically configured matrix for tests, master branch options, combined workflow for releases

catalyst / catalyst-moodle-workflows

4 stars 9 forks source link

Concurrency, dynamically configured matrix for tests, master branch options, combined workflow for releases #25

Closed keevan closed 2 years ago

keevan commented 2 years ago

TODO:

[ ] Update README to reflect new changes.
[ ] Squash commits, removing previous manual changes to different workflow groups

After updating the ci for plugins using the workflows to ../workflows/ci.yml instead of a dynamically set group, it should be fine to remove the unused workflow files after that.

Resolves #16 #22 #18 #9

keevan commented 2 years ago

FYI, the vendor files was because I tried adding the yaml php extension, but it took ages to install (at least locally) compared to a much quicker run of the php base setup without the extension. So I opted for a known yaml parser (symfony), and although the package could be installed via composer install, I've opted to keep it as source for now to avoid the step of installing composer before running the prep job. This can be changed later on if needed.