Skip to content

fix(hugo2confluence): replace confluence markdown converter [ENG-4650]#789

Merged
egibs merged 1 commit intochainguard-dev:mainfrom
stevebeattie:hugo2confluence-replace_markdown_converter
Mar 5, 2026
Merged

fix(hugo2confluence): replace confluence markdown converter [ENG-4650]#789
egibs merged 1 commit intochainguard-dev:mainfrom
stevebeattie:hugo2confluence-replace_markdown_converter

Conversation

@stevebeattie
Copy link
Copy Markdown
Member

The hugo2confluence mirror script would use the confluence built in converter for TinyMCE, the legacy editor for Confluence. However, our Confluence has moved to the Fabric editor which is stricter about format validation, and the old editor was producing unclosed <li> objects and loose nesting.

Fix this by switching to using the gomarkdown/markdown golang library to do the conversion to XHTML for ingestion by Confluence. This addresses failures that were being seen in the chainweb-mirror workflwow for the engineering wiki that was preventing the EmiritOSS wiki page from being created in Confluence.

The hugo2confluence mirror script would use the confluence built in
converter for TinyMCE, the legacy editor for Confluence. However, our
Confluence has moved to the Fabric editor which is stricter about format
validation, and the old editor was producing unclosed `<li>` objects and
loose nesting.

Fix this by switching to using the gomarkdown/markdown golang library to
do the conversion to XHTML for ingestion by Confluence. This addresses
failures that were being seen in the chainweb-mirror workflwow for the
engineering wiki that was preventing the EmiritOSS wiki page from being
created in Confluence.

Signed-off-by: Steve Beattie <steve.beattie@chainguard.dev>
@stevebeattie stevebeattie requested review from egibs and jdolitsky March 5, 2026 05:32
Comment thread hugo2confluence/main.go
@egibs egibs added this pull request to the merge queue Mar 5, 2026
Merged via the queue into chainguard-dev:main with commit e9290fa Mar 5, 2026
7 checks passed
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