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.

Sign up for GitHub

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

Jump to bottom

[bug] Comments in OCaml quoted string literals #1627

Open
ianlewis opened this issue Feb 11, 2025 · 0 comments
Open

[bug] Comments in OCaml quoted string literals #1627

ianlewis opened this issue Feb 11, 2025 · 0 comments
Assignees
@ianlewis
Labels
bug Something isn't working
Milestone
v1.0.0

Comments

@ianlewis
Copy link
Owner

ianlewis commented Feb 11, 2025 •
edited
Loading

TODO comments inside OCaml quoted string literals are picked up by todos.

For example:

let nested = {ext|(* TODO: foo *)|ext};

This shouldn't be detected as a TODO comment since it's in a string literal but it is:

$ todos
test.ml:1:(* TODO: foo *)
@ianlewis ianlewis added the enhancement New feature or request label Feb 11, 2025
@ianlewis ianlewis self-assigned this Feb 11, 2025
@ianlewis ianlewis added this to the v1.0.0 milestone Feb 11, 2025
@ianlewis ianlewis added bug Something isn't working and removed enhancement New feature or request labels Feb 11, 2025
@ianlewis ianlewis changed the title [feature] OCaml quoted string literals [bug] Comments in OCaml quoted string literals Feb 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ianlewis ianlewis

Labels
bug Something isn't working
Projects
None yet
Milestone
v1.0.0
Development

No branches or pull requests
1 participant
@ianlewis