Skip to content

Migrate to Marked-mustache in markdown.js instead of inlined code for {} and {{}} markdown extensions#4444

Open
dbolack-ab wants to merge 7 commits intonaturalcrit:masterfrom
dbolack-ab:marked-mustache
Open

Migrate to Marked-mustache in markdown.js instead of inlined code for {} and {{}} markdown extensions#4444
dbolack-ab wants to merge 7 commits intonaturalcrit:masterfrom
dbolack-ab:marked-mustache

Conversation

@dbolack-ab
Copy link
Copy Markdown
Collaborator

Description

Migrates the mustache extensions from markdown.js to their own npm module.

Make sure the migrated code is up to date in the module before merging!

Related Issues or Discussions

#4035

Reviewer Checklist

*Reviewers, refer to this list when testing features, or suggest new items *

  • Verify new features are functional
    • Ensure tests still test

@dbolack-ab dbolack-ab added cleanup Cleaning up code for legibility, style, ease-of-use, etc. 🔍 R0 - Needs first review 👀 PR ready but has not been reviewed 🎃 Hacktoberfest Intended for Hacktoberfest credit labels Oct 3, 2025
@calculuschild calculuschild temporarily deployed to homebrewery-pr-4444 October 4, 2025 17:28 Inactive
@dbolack-ab dbolack-ab changed the title Marked mustache Migrate to Marked-mustache in markdown.js instead of inlined code for {} and {{}} markdown extensions Oct 11, 2025
@dbolack-ab
Copy link
Copy Markdown
Collaborator Author

@5e-Cleric
Copy link
Copy Markdown
Member

Everything seems to work, can you fix the conflicts so i can approve this and merge?

@5e-Cleric 5e-Cleric added 🔍 R3 - Reviewed - Awaiting Fixes 🔧 PR is okayed but needs fixes before merging and removed 🔍 R0 - Needs first review 👀 PR ready but has not been reviewed labels Nov 10, 2025
@5e-Cleric 5e-Cleric moved this from Waiting for Calc's First Review to Under Review in @calculuschild's backlog Nov 10, 2025
@calculuschild
Copy link
Copy Markdown
Member

@5e-Cleric the external package itself needs some tweaks (and accompanying version bump) before this can merge

@5e-Cleric 5e-Cleric added the blocked Waiting on a dependency, other feature, etc., first label Nov 12, 2025
@dbolack-ab
Copy link
Copy Markdown
Collaborator Author

@5e-Cleric the external package itself needs some tweaks (and accompanying version bump) before this can merge

marked-modules-transitional/marked-mustache#1 should take care of this.

@5e-Cleric
Copy link
Copy Markdown
Member

5e-Cleric commented Feb 4, 2026

Will close #4035

@5e-Cleric 5e-Cleric linked an issue Feb 4, 2026 that may be closed by this pull request
12 tasks
@dbolack-ab dbolack-ab added the Orphan This project's original dev is not going to finish this, feel free to pick it up label Mar 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

blocked Waiting on a dependency, other feature, etc., first cleanup Cleaning up code for legibility, style, ease-of-use, etc. 🎃 Hacktoberfest Intended for Hacktoberfest credit Orphan This project's original dev is not going to finish this, feel free to pick it up 🔍 R3 - Reviewed - Awaiting Fixes 🔧 PR is okayed but needs fixes before merging

Projects

Status: Under Review

Development

Successfully merging this pull request may close these issues.

Break markdown extensions in markdown.js out into their own npm modules

4 participants