Skip to content

ci: install asciidoc in deploy workflow before building HTML docs - #17

Draft
hub966 with Copilot wants to merge 2 commits into
masterfrom
copilot/fix-deploy-job-failure
Draft

ci: install asciidoc in deploy workflow before building HTML docs#17
hub966 with Copilot wants to merge 2 commits into
masterfrom
copilot/fix-deploy-job-failure

Conversation

Copilot AI commented Aug 6, 2026

Copy link
Copy Markdown

The deploy GitHub Actions job was failing because asciidoc is not pre-installed on ubuntu-latest runners, causing make -C Documentation html to error out immediately.

Changes

  • .github/workflows/static.yml: Added an Install asciidoc step using apt-get prior to the HTML documentation build step
- name: Install asciidoc
  run: sudo apt-get update -q && sudo apt-get install -y asciidoc
- name: Build HTML documentation
  run: make -C Documentation html

Copilot AI lite review requested due to automatic review settings August 6, 2026 14:40

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot wasn't able to review any files in this pull request.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Co-authored-by: hub966 <205647957+hub966@users.noreply.github.com>
Copilot AI review requested due to automatic review settings August 6, 2026 14:42
Copilot AI changed the title [WIP] Fix failing GitHub Actions job deploy ci: install asciidoc in deploy workflow before building HTML docs Aug 6, 2026
Copilot AI requested a review from hub966 August 6, 2026 14:43

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

Suppressed comments (1)

.github/workflows/static.yml:37

  • ci/install-dependencies.sh already installs the documentation toolchain (including asciidoc) when jobname=Documentation (see ci/install-dependencies.sh:166-172). Adding a separate apt-get install step duplicates dependency management and may drift/miss other doc prerequisites. Prefer setting jobname for the existing dependency step and removing this extra install step.
      - name: Install dependencies
        run: ci/install-dependencies.sh
      - name: Install asciidoc
        run: sudo apt-get update -q && sudo apt-get install -y asciidoc

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.

3 participants