Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: escape underscores that are within latex equations #137

Merged
merged 4 commits into from
Jan 29, 2025

Conversation

rateixei
Copy link
Contributor

@rateixei rateixei commented Jan 29, 2025

Add regex so that expressions that are enclosed between two dollar signs do not have their underscores modified. Extra code block is added to handle standalone equations export (adding $$ equation $$).

Copy link

mergify bot commented Jan 29, 2025

Merge Protections

Your pull request matches the following merge protections and will not be merged until they are valid.

🟢 Enforce conventional commit

Wonderful, this rule succeeded.

Make sure that we follow https://www.conventionalcommits.org/en/v1.0.0/

  • title ~= ^(fix|feat|docs|style|refactor|perf|test|build|ci|chore|revert)(?:\(.+\))?(!)?:

@rateixei rateixei force-pushed the rtdl/escape_latex_underscores branch from feb93f3 to 60694bb Compare January 29, 2025 09:56
@rateixei rateixei changed the title Escape underscores that are within latex equations feat: escape underscores that are within latex equations Jan 29, 2025
Signed-off-by: Rafael Teixeira de Lima <[email protected]>
Signed-off-by: Rafael Teixeira de Lima <[email protected]>
Signed-off-by: Rafael Teixeira de Lima <[email protected]>
@rateixei rateixei force-pushed the rtdl/escape_latex_underscores branch from 3d618df to a483d26 Compare January 29, 2025 13:33
@PeterStaar-IBM PeterStaar-IBM merged commit 0d5cd11 into main Jan 29, 2025
8 checks passed
@PeterStaar-IBM PeterStaar-IBM deleted the rtdl/escape_latex_underscores branch January 29, 2025 13:49
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