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

add % to the regex for latex #815

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

HernandoR
Copy link

add % to regex for tex files, untested

add % for tex files
@HernandoR HernandoR changed the title fix #737 add % to the regex for latex Jan 22, 2024
@HernandoR
Copy link
Author

Noticed that this week will also include % in the equation input part i.e. $ \% todo$ will be matched.

and if merged, a=b%todo in auto languages will be matched. Even though % is not a comment symbol in python.
This behavior aligned with other languages: # todo in markdown will also be matched. Even though # is not a comment symbol in markdown.

@GitMensch
Copy link

GitMensch commented May 6, 2024

Should be closed as duplicate of #568 which notes:

Hi, sorry to disappoint you, but this won't get merged. I no longer update the default regex as it causes too many issues for people who use the original default. Every time I've done it (after convincing myself it wouldn't have any effect) I've had to back it out again.

The default is really only intended to be a starting point - it would be impossible for the default to work for everybody.

The wiki contains some example configurations for LaTeX: https://github.com/Gruntfuggly/todo-tree/wiki/Configuration-Examples#latex

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