Skip to content

feat: consolidate tool-specific rake workflows into generic-rake.yml - #259

Merged
ronaldtse merged 1 commit into
mainfrom
consolidate-rake-workflows
Apr 15, 2026
Merged

feat: consolidate tool-specific rake workflows into generic-rake.yml#259
ronaldtse merged 1 commit into
mainfrom
consolidate-rake-workflows

Conversation

@ronaldtse

Copy link
Copy Markdown
Contributor

Summary

  • Add setup-tools, submodules, private-fonts, private-fonts-username, and choco-cache inputs to generic-rake.yml, enabling it to replace 4 near-identical tool-specific workflows
  • Update 5 Cimas templates to call generic-rake.yml with appropriate inputs instead of inkscape-rake, graphviz-rake, libreoffice-rake, xml2rfc-rake
  • Old workflow files are kept unchanged — they will be deleted in a follow-up PR after Cimas sync verifies all downstream repos pass CI

Migration mapping

Cimas template Old workflow New setup-tools
inkscape/rake.yml inkscape-rake.yml inkscape,ghostscript
inkscape/rake-daily.yml inkscape-rake.yml inkscape,ghostscript
graphviz/rake.yml graphviz-rake.yml graphviz
libreoffice/rake.yml libreoffice-rake.yml libreoffice
xml2rfc/rake.yml xml2rfc-rake.yml xml2rfc

Bug fix included

libreoffice-rake.yml:36 uses matrix.rubygems instead of matrix.ruby.rubygems — migration to generic-rake.yml fixes this automatically.

Test plan

  • actionlint passes on all modified files
  • Cimas sync to downstream repos (post-merge)
  • Verify CI passes on: metanorma, isodoc, reverse_adoc, metanorma-ietf
  • Follow-up PR to delete old workflow files

Add setup-tools, submodules, private-fonts, choco-cache inputs to
generic-rake.yml so it can replace inkscape-rake, graphviz-rake,
libreoffice-rake, and xml2rfc-rake. Update cimas templates to call
generic-rake.yml with appropriate tool configurations.

Old workflow files are kept unchanged until all downstream repos are
migrated via Cimas sync. Also fixes libreoffice-rake bug where
matrix.rubygems was used instead of matrix.ruby.rubygems.
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.

1 participant