Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

mdbook-mermaid is missing from docs build+deploy workflow #513

Closed
jeckersb opened this issue May 6, 2024 · 0 comments · Fixed by #514
Closed

mdbook-mermaid is missing from docs build+deploy workflow #513

jeckersb opened this issue May 6, 2024 · 0 comments · Fixed by #514
Assignees

Comments

@jeckersb
Copy link
Contributor

jeckersb commented May 6, 2024

Causing docs publishing to fail

Run cd docs && mdbook build
2024-05-06 17:38:41 [INFO] (mdbook::book): Book building has started
2024-05-06 17:38:41 [WARN] (mdbook::preprocess::cmd): The command wasn't found, is the "mermaid" preprocessor installed?
2024-05-06 17:38:41 [WARN] (mdbook::preprocess::cmd): 	Command: mdbook-mermaid
2024-05-06 17:38:41 [INFO] (mdbook::book): Running the html backend
Error: -06 17:38:41 [ERROR] (mdbook::utils): Error: Rendering failed
Error: -06 17:38:41 [ERROR] (mdbook::utils): 	Caused By: Unable to copy across additional CSS and JS
Error: -06 17:38:41 [ERROR] (mdbook::utils): 	Caused By: Unable to copy /home/runner/work/bootc/bootc/docs/mermaid.min.js to /home/runner/work/bootc/bootc/docs/book/mermaid.min.js
Error: -06 17:38:41 [ERROR] (mdbook::utils): 	Caused By: No such file or directory (os error 2)
Error: Process completed with exit code 101.

The mdbook-mermaid requirement was added in #503 but we forgot to add it to the docs workflow.

@jeckersb jeckersb self-assigned this May 6, 2024
jeckersb added a commit to jeckersb/bootc that referenced this issue May 6, 2024
Fixes containers#513

Signed-off-by: John Eckersberg <jeckersb@redhat.com>
lukewarmtemp pushed a commit to lukewarmtemp/bootc that referenced this issue May 9, 2024
Fixes containers#513

Signed-off-by: John Eckersberg <jeckersb@redhat.com>
lukewarmtemp pushed a commit to lukewarmtemp/bootc that referenced this issue May 9, 2024
Fixes containers#513

Signed-off-by: John Eckersberg <jeckersb@redhat.com>
lukewarmtemp pushed a commit to lukewarmtemp/bootc that referenced this issue May 22, 2024
Fixes containers#513

Signed-off-by: John Eckersberg <jeckersb@redhat.com>
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 a pull request may close this issue.

1 participant