Skip to content

Comments

Add support for code in blockquotes in custom renderer#111

Merged
andysteadbbc merged 1 commit intomainfrom
fix-code-blocks-in-blockquotes
Feb 6, 2026
Merged

Add support for code in blockquotes in custom renderer#111
andysteadbbc merged 1 commit intomainfrom
fix-code-blocks-in-blockquotes

Conversation

@andysteadbbc
Copy link
Contributor

🧐 What?

Found an error when code blocks were added in a certain way with a NOTE github action blockquote

{
    "errorType": "Error",
    "errorMessage": "Token with \"paragraph\" type was not found.\nPlease report this to https://github.com/markedjs/marked.",
    "stack": [
        "Error: Token with \"paragraph\" type was not found.",
        "Please report this to https://github.com/markedjs/marked.",
        "    at $k.parseInline (/var/task/node_modules/@bbc/morty-docs/build/index.js:476:1068)",
        "    at /var/task/node_modules/@bbc/morty-docs/build/index.js:477:230494",
        "    at Array.forEach (<anonymous>)",
        "    at Rh.blockquote (/var/task/node_modules/@bbc/morty-docs/build/index.js:477:230281)",
        "    at s.<computed> [as blockquote] (/var/task/node_modules/@bbc/morty-docs/build/index.js:476:3332)",
        "    at $k.parse (/var/task/node_modules/@bbc/morty-docs/build/index.js:475:1975)",
        "    at parse (/var/task/node_modules/@bbc/morty-docs/build/index.js:475:1296)",
        "    at tm.parse (/var/task/node_modules/@bbc/morty-docs/build/index.js:476:5638)",
        "    at Function.pt [as parse] (/var/task/node_modules/@bbc/morty-docs/build/index.js:477:239)",
        "    at dC (/var/task/node_modules/@bbc/morty-docs/build/index.js:477:230934)"
    ]
}

🛠 Fix

Add a check for this in the blcokquote custom renderer

@andysteadbbc andysteadbbc requested a review from a team as a code owner February 6, 2026 10:02
@andysteadbbc andysteadbbc merged commit 8622ea7 into main Feb 6, 2026
4 checks passed
@andysteadbbc andysteadbbc deleted the fix-code-blocks-in-blockquotes branch February 6, 2026 10:15
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