Skip to content

Fix callout build error from escaped shortcode syntax - #116

Merged
leecalcote merged 1 commit into
masterfrom
fix/callout-build-error
Apr 2, 2026
Merged

Fix callout build error from escaped shortcode syntax#116
leecalcote merged 1 commit into
masterfrom
fix/callout-build-error

Conversation

@simihablo

Copy link
Copy Markdown

Summary

  • Fix Hugo build error in the content-formatting-examples page where escaped shortcode delimiters ({{</* */>}}) inside a hextra/callout body caused markdownify to fail with "shortcode has no name"
  • Hugo resolves escaped shortcode syntax to literal delimiters in .InnerDeindent, which then breaks markdownify processing
  • Reworded the callout text to describe the delimiter syntax without using literal shortcode notation

Test plan

  • Verify the Hugo build completes without errors: hugo --gc --minify -D
  • Verify the content-formatting-examples page renders the Hextra callout correctly
  • Confirm no other shortcode examples are affected

Hugo resolves escaped shortcode delimiters (e.g. {{</* */>}}) to
literal {{< >}} inside .InnerDeindent, which then causes markdownify
to fail with "shortcode has no name". Reword the callout text to avoid
embedded shortcode delimiter literals.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the content-formatting-examples documentation to avoid a Hugo build failure caused by escaped shortcode delimiters being included inside a hextra/callout body that is processed via markdownify.

Changes:

  • Reworded the informational hextra/callout text to describe shortcode delimiter usage without including literal escaped shortcode syntax that can break markdownify.

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

@leecalcote
leecalcote merged commit db6ca7f into master Apr 2, 2026
5 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants