Skip to content

ci: simplify docker workflow using actions-mn actions#40

Open
ronaldtse wants to merge 3 commits into
mainfrom
simplify-docker-workflow
Open

ci: simplify docker workflow using actions-mn actions#40
ronaldtse wants to merge 3 commits into
mainfrom
simplify-docker-workflow

Conversation

@ronaldtse
Copy link
Copy Markdown
Contributor

Summary

  • Replace metanorma/ci/docker-gem-install@main (failing with bundler exit code 5) with actions-mn/cache@v1 + actions-mn/site-gen@v1
  • Remove manual actions/cache@v4 orchestration — actions-mn/cache@v1 handles manifest-based site caching internally
  • Remove separate "Upload cached Metanorma XML" conditional step — always upload _site as artifact
  • Drop metanorma --version debug step (not essential)

What changed

Before After
metanorma/ci/docker-gem-install@main Eliminated
Manual actions/cache@v4 with hash key actions-mn/cache@v1
actions-mn/build-and-publish@v2 actions-mn/site-gen@v1
Conditional artifact upload for cache hit/miss Single actions/upload-artifact@v4

Fixes the bundler exit code 5 failure on the build job.

ronaldtse added 3 commits May 11, 2026 11:04
Replace metanorma/ci/docker-gem-install@main (which fails with bundler
exit code 5) and manual actions/cache@v4 logic with actions-mn/cache@v1
and actions-mn/site-gen@v1.

The actions-mn actions handle font caching, manifest-based site caching,
and metanorma site generation internally, eliminating the need for a
separate bundler install step.
The actions-mn/cache@v1 is a composite action that uses
actions/github-script internally, which fails inside Docker containers
with ERR_PACKAGE_PATH_NOT_EXPORTED.

Keep actions/cache@v4 for site output caching, and use
actions-mn/site-gen@v1 for the build step.
Explain how the release workflow uses normalized document identifiers
from RXL metadata (not source directory names) for pattern matching.
Add workflow_dispatch inputs for include-pattern and force.
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