Add ci.yml workflow to modules

[emteknetnz]

ACs

• Add workflow to all supported silverstripe modules to use ci.yml - NOTE: this needs to target the oldest supported branches and be merged-up, not just the default branches
• Best effort to maintain any custom logic within .travis.yml files
• Maintain travis ci support for a while to validate things work as expected
• Update all module readme status badges from travis-ci to github actions default branch
• Ensure workflows do not run on forks, or at least scheduled workflows do not run on forks
• Workflows are green

ACs Round 2 PRs

• Remove .travis.yml files
• In ci.yml and keepalive.yml (and readmes gha-ci, gha-keepalive and gha-update-js) change startsWith(github.repository, '<account>/') to github.repository_owner == '<account>'
• Some travis badges were not updated e.g. silvestripe/silversripe-postgresql - the initial $find in module-standardiser was too strict
• Remove travis badges and add new CI badges to https://github.com/silverstripe/supported-modules
  • Use the updated table to populate the supported modules table in silverstripe.org

Notes

• Usually makes more sense to just hardcode things in ci.yml rather than trying to make ci.yml move configurable
• Workarounds to make workflows not run on forks - https://github.com/microsoft/TypeScript/pull/48693/files - https://gh.neting.ccmunity/t/do-not-run-cron-workflows-in-forks/17636
• When creating a pull-request to add ci.yml to a repo, it will run on pull-request BUT ONLY if there are free github action worker available. It will not queue the workflow like you would expect. This is certainly inconvenient.
• I have validated that account that have maxed out their github actions workers will still queue up ci runs, provided there is a ci.yml defined on the repo - i.e. you cannot bypass ci.yml by just spamming pull-requests. Test here.

Custom .travis files

• assets - PHP_COVERAGE_SUITE=assets .. to stop travis timing out after 10 minutes not required
• graphql (both 3 & 4)
• framework - custom matrix cos phpcoverage was giving weird results on travis
• auditor - mfa + session_manager custom

Custom .travis files, though probably does not need to be custom

• recipe-core - requires themes simple + admin + versioned - but there's no behat
• vendor-plugin - may work out of the box with ci.yml
• sharedraftcontent
• userforms
• spellcheck - apt packages (ci.yml has this hardcoded)

Graphql3

• Graphql4 will install by default
• Add explicitly add a graphql3 behat 'extra_job' for asset-admin, versioned-admin and elemental

missing .travis.yml / ancient non-shared .travis

• mink-facebook-web-driver (only unit testing thing in vendor folder, should probably just remove)
• recipe-plugin (no current CI, no unit tests)
• recipe-testing (no current CI, no unit tests, no code)
• serve (has unit tests)
• testsession (no current CI, no unit tests)

.travis files where REQUIRE_EXTRA should be moved to composer.json require-dev

Note: keep silverstripe/frameworktest as a requirement in module ci.yml, rather than composer require-dev, because it should eventually get split to move the 'behat test models' to a separate module - issue. If/when that happens, consider moving the requirement of the new module to require-dev

• elemental-bannerblock require elemental
• recipe-form-building - requies gridfieldqueuedexport + queuedjobs
• blog - requires widgets, comments, content-widgets
• comments - requires ezyang/htmlpurifier
• contentreview - requires symbiote/silverstripe-queuedjobs
• externallinks - requires symbiote/silverstripe-queuedjobs
• fulltextsearch - requires queuedjobs and subsites
• recipe-blog - requires gridfieldqueuedexport, queuedjobs
• reciple-ccl - requires versioned
• recipe-solr-search - requires cwp/cwp, cwp/cwp-core, cwp starter theme
• securityreport - requires subsites
• sitewidgecontent-report - requies subsites, taxonomy, contentreview
• totp-authenticator - requires webautn-authenticator - other things are sorted in ci.yml

PRs - CI

• NEW Add phpcoverage_force_off input gha-generate-matrix#3
• NEW Add phpcoverage_force_off input #20
• FIX Quote input yaml strings gha-generate-matrix#5
• FIX PDO transaction in PHP 8, add GitHub Actions CI silverstripe-framework#10360
• FIX Update getInstallerVersion gha-generate-matrix#6
• FIX Detect COMPOSER_ROOT_VERSION on push events #23
• FIX Use parent_branch gha-generate-matrix#7
• FIX Handle zero-match gha-generate-matrix#8
• FIX Handle zero-match #24
• FIX Use max of PHP 8.0 for x.10 recipes gha-generate-matrix#10
• DOC Only run crons on base account #22
• DOC Only run crons on base account gha-update-js#4
• DOC Only run crons on base account gha-keepalive#4
• FIX Only add extension if elemental module exists silverstripe-frameworktest#109
• FIX Use correct repo name for silverstripe-spellcheck #26
• FIX Handle missing tables in postgres on flush silverstripe-graphql#483
• FIX Increase health retries #27
• FIX Allow installing guzzle 6 compatible version of cow #29
• FIX Ensure yarn scripts exist before running them gha-run-tests#3
• FIX Move parent branch logic gha-generate-matrix#13
• FIX Set fetch-depth for parent-branch #31
• FIX Set fetch-depth for parent branch gha-generate-matrix#14
• FIX Move fetch-depth to gha-generate-matrix #32
• FIX Use correct repository for postgres #34

PRs - Modules

• MNT Use GitHub Actions CI silverstripe-versioned-admin#241
• MNT Use GitHub Actions CI silverstripe-versioned#365
• MNT Use GitHub Actions CI silverstripe-siteconfig#122
• MNT Use GitHub Actions CI silverstripe-reports#149
• MNT Use GitHub Actions CI silverstripe-tagfield#213
• MNT Use GitHub Actions CI comment-notifications#45
• MNT Use GitHub Actions CI cwp#310
• MNT Use GitHub Actions CI cwp-pdfexport#25
• MNT Use GitHub Actions CI cwp-search#44
• MNT Use GitHub Actions CI cwp-starter-theme#210
• MNT Use GitHub Actions CI cwp-watea-theme#148
• MNT Use GitHub Actions CI silverstripe-simple#70
• MNT Use GitHub Actions CI doorman#41
• MNT Use GitHub Actions CI silverstripe-serve#32
• MNT Use GitHub Actions CI silverstripe-graphql-devtools#43
• MNT Use GitHub Actions CI webpack-config#50
• MNT Use GitHub Actions CI silverstripe-akismet#42
• MNT Use GitHub Actions CI silverstripe-auditor#44
• MNT Use GitHub Actions CI silverstripe-admin#1319
• MNT Use GitHub Actions CI silverstripe-asset-admin#1277
• MNT Use GitHub Actions CI silverstripe-assets#500
• MNT Use GitHub Actions CI silverstripe-blog#680
• MNT Use GitHub Actions CI silverstripe-campaign-admin#235
• MNT Use GitHub Actions CI silverstripe-ckan-registry#269
• MNT Use GitHub Actions CI silverstripe-cms#2745
• MNT Use GitHub Actions CI silverstripe-config#61
• MNT Use GitHub Actions CI silverstripe-errorpage#68
• MNT Use GitHub Actions CI silverstripe-graphql#481 (graphql 3)
• MNT Use GitHub Actions CI silverstripe-graphql#482 (graphql 4)
• MNT Use GitHub Actions CI silverstripe-comments#342
• MNT Use GitHub Actions CI silverstripe-content-widget#31
• MNT Use GitHub Actions CI silverstripe-contentreview#165
• MNT Use GitHub Actions CI silverstripe-crontask#67
• MNT Use GitHub Actions CI silverstripe-documentconverter#31
• MNT Use GitHub Actions CI silverstripe-elemental#989
• MNT Use GitHub Actions CI silverstripe-userforms#1156
• MNT Use GitHub Actions CI silverstripe-widgets#191
• MNT Use GitHub Actions CI silverstripe-mfa#459
• MNT Use GitHub Actions CI silverstripe-totp-authenticator#95
• MNT Use GitHub Actions CI silverstripe-webauthn-authenticator#115
• MNT Use GitHub Actions CI silverstripe-login-forms#116
• MNT Use GitHub Actions CI silverstripe-security-extensions#44
• MNT Use GitHub Actions CI silverstripe-upgrader#200
• MNT Use GitHub Actions CI silverstripe-versionfeed#71
• MNT Use GitHub Actions CI vendor-plugin#54
• MNT Use GitHub Actions CI symbiote/silverstripe-advancedworkflow#462
• MNT Use GitHub Actions CI symbiote/silverstripe-gridfieldextensions#345
• MNT Use GitHub Actions CI symbiote/silverstripe-multivaluefield#82
• MNT Use GitHub Actions CI symbiote/silverstripe-queuedjobs#378
• MNT Use GitHub Actions CI cow#217
• MNT Use GitHub Actions CI eslint-config#10
• MNT Use GitHub Actions CI MinkFacebookWebDriver#19
• MNT Use GitHub Actions CI recipe-testing#9
• MNT Use GitHub Actions CI silverstripe-behat-extension#215
• MNT Use GitHub Actions CI silverstripe-postgresql#130
• MNT Use GitHub Actions CI silverstripe-selectupload#39
• MNT Use GitHub Actions CI silverstripe-sitewidecontent-report#52
• MNT Use GitHub Actions CI recipe-ccl#5
• MNT Use GitHub Actions CI cwp-agencyextensions#74
• MNT Use GitHub Actions CI silverstripe-session-manager#112
• MNT Use GitHub Actions CI silverstripe-fulltextsearch#327
• MNT Use GitHub Actions CI silverstripe-testsession#75
• MNT Use GitHub Actions CI recipe-solr-search#11
• MNT Use GitHub Actions CI recipe-reporting-tools#28
• MNT Use GitHub Actions CI recipe-form-building#19
• MNT Use GitHub Actions CI recipe-blog#33
• MNT Use GitHub Actions CI silverstripe-ldap#41
• MNT Use GitHub Actions CI silverstripe-externallinks#81
• MNT Use GitHub Actions CI silverstripe-securityreport#55
• MNT Use GitHub Actions CI recipe-plugin#23
• MNT Use GitHub Actions CI recipe-core#73
• MNT Use GitHub Actions CI silverstripe-gridfieldqueuedexport#67
• MNT Use GitHub Actions CI silverstripe-textextraction#74
• MNT Use GitHub Actions CI silverstripe-taxonomy#80
• MNT Use GitHub Actions CI silverstripe-subsites#482
• MNT Use GitHub Actions CI silverstripe-spellcheck#52
• MNT Use GitHub Actions CI silverstripe-spamprotection#83
• MNT Use GitHub Actions CI silverstripe-sharedraftcontent#170
• MNT Use GitHub Actions CI silverstripe-segment-field#67
• MNT Use GitHub Actions CI silverstripe-restfulserver#99
• MNT Use GitHub Actions CI silverstripe-registry#68
• MNT Use GitHub Actions CI recipe-services#20
• MNT Use GitHub Actions CI recipe-content-blocks#23
• MNT Use GitHub Actions CI recipe-collaboration#17
• MNT Use GitHub Actions CI recipe-cms#57
• MNT Use GitHub Actions CI recipe-authoring-tools#20
• MNT Use GitHub Actions CI silverstripe-realme#88
• MNT Use GitHub Actions CI silverstripe-mimevalidator#54
• MNT Use GitHub Actions CI silverstripe-lumberjack#112
• MNT Use GitHub Actions CI silverstripe-iframe#59
• MNT Use GitHub Actions CI silverstripe-hybridsessions#73
• MNT Use GitHub Actions CI silverstripe-html5#37
• MNT Use GitHub Actions CI silverstripe-environmentcheck#79
• MNT Use GitHub Actions CI silverstripe-elemental-fileblock#30
• MNT Use GitHub Actions CI silverstripe-elemental-bannerblock#88
• MNT Use GitHub Actions CI cwp-core#108
• MNT Use GitHub Actions CI silverstripe-installer#326 (do last, most issues will be in individual modules)
• MNT Use GitHub Actions CI recipe-kitchen-sink#14 (do last, most issues will be in individual modules)

[emteknetnz]

We'll use this to create all the initial pull-requests - https://github.com/emteknetnz/module-standardiser

It creates PRs that look like this silverstripe/silverstripe-tagfield#212

All the cron jobs are scheduled at varying times throughout the week so we don't clog things up

For any custom logic, we'll add this manually after the initial pull-requests have been created

[GuySartorelli]

Looks like a lot of supported modules are missing from the PR list:

• bringyourownideas/silverstripe-maintenance
• bringyourownideas/silverstripe-composer-update-checker
• bringyourownideas/silverstripe-composer-security-checker
• colymba/gridfield-bulk-editing-tools
• cwp/cwp-core
• dnadesign/silverstripe-elemental-subsites
• dnadesign/silverstripe-elemental-userforms
• hafriedlander/phockito
• hafriedlander/silverstripe-phockito
• lekoala/silverstripe-debugbar
• silverstripe/silverstripe-activedirectory
• silverstripe/elemental-bannerblock
• silverstripe/elemental-fileblock
• silverstripe/environmentcheck
• silverstripe/externallinks
• silverstripe/fulltextsearch
• silverstripe/gridfieldqueuedexport
• silverstripe/html5
• silverstripe/hybridsessions

Probably more too - I stopped checking when I got that far.

[maxime-rainville]

Didn't quite manage to get all the builds green, but these are mostly failures that just got highlighted by the new build system. We'll create a follow up card.

[maxime-rainville]

Didn't quite manage to get all the builds green, but these are mostly failures that just got highlighted by the new build system. We'll create a follow up card.