feat: add backtick standardization to clean up markdown-code-runner t… #27
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…ags in output files and protect them when doing an in-place replacement. Needed because, systems like mkdocs, pandoc are unable to parse fenced blocks if they have extra arguments other than the language to be highlighted.
Owner
|
Thanks for the excellent PR! The backtick standardization feature works perfectly and addresses the mkdocs/pandoc compatibility issue. I've simplified the implementation while maintaining 100% identical functionality. |
- Consolidate _extract_backtick_options from 22 to 12 lines using single regex - Remove START_MARKERS list and inline marker checking logic - Merge _process_backticks_start into _process_start_markers - Replace complex --backtick-standardize/--force-overwrite with simple --no-backtick-standardize - Remove unnecessary .gitignore file Maintains 100% identical behavior with ~50 fewer lines of code. All 30 tests pass with 100% coverage.
basnijholt
force-pushed
the
backtick-output-standardazie
branch
from
a82dd89 to
40584f9
Compare
Contributor
Author
|
Thanks.
Appreciate the collaboration.
I am using this as a dependency in my project.
Thank you for the useful implementation.
Cheers.
…On Tue, Jul 22, 2025 at 12:22 PM Bas Nijholt ***@***.***> wrote:
*basnijholt* left a comment (basnijholt/markdown-code-runner#27)
<#27 (comment)>
https://github.com/basnijholt/markdown-code-runner/releases/tag/v2.3.0 🎉
—
Reply to this email directly, view it on GitHub
<#27 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABMPQJ2DU72OYAXUGDMDAML3JXNSXAVCNFSM6AAAAACBLG42DKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZTCMBRGMZDIMBSGQ>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
Owner
|
Awesome! Which project is that (if it is OSS?) |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
#26
Needed because, systems like mkdocs, pandoc are unable to parse fenced blocks if they have extra arguments other than the language to be highlighted.