Skip to content

Separate into reusable workflows so we can schedule CI for release branches#3570

Merged
jcoyne merged 3 commits intomainfrom
weekly_ci_run
Apr 18, 2025
Merged

Separate into reusable workflows so we can schedule CI for release branches#3570
jcoyne merged 3 commits intomainfrom
weekly_ci_run

Conversation

@maxkadel
Copy link
Copy Markdown
Contributor

@maxkadel maxkadel commented Apr 17, 2025

See this workflow run for an example of what will happen if one of the release-8.x CI runs fails.

In order to re-use workflows, separated existing configuration into reusable workflows.

TODO: Allow more of the matrix to be configurable, especially the "include" stanza. Once the release-7.x branch is green, this will be needed to add that branch to this configuration.

It could also be worth it to put these actions into their own repo in the projectblacklight namespace, and try to configure some of the plugins to use them as well, which could help with compatibility snarls, but that's a bigger project.

@maxkadel maxkadel changed the title Commit lint.yml so we can reference it Separate into reusable workflows so we can schedule CI for release branches Apr 17, 2025
In order to re-use workflows, separated existing configuration into reusable workflows.

TODO: Allow more of the matrix to be configurable, especially the "include" stanza.
@maxkadel maxkadel marked this pull request as ready for review April 17, 2025 20:24
@maxkadel maxkadel marked this pull request as draft April 18, 2025 13:41
@maxkadel maxkadel marked this pull request as ready for review April 18, 2025 13:53
Copy link
Copy Markdown
Member

@jcoyne jcoyne left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great job @maxkadel

@jcoyne jcoyne merged commit abd7a2a into main Apr 18, 2025
20 checks passed
@jcoyne jcoyne deleted the weekly_ci_run branch April 18, 2025 14:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants