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

existence rule skips match in RST #831

Closed
1 task done
mikemckiernan opened this issue May 17, 2024 · 2 comments
Closed
1 task done

existence rule skips match in RST #831

mikemckiernan opened this issue May 17, 2024 · 2 comments

Comments

@mikemckiernan
Copy link

mikemckiernan commented May 17, 2024

Check for existing issues

  • Completed

Environment

Ubuntu 20.04 in WSL2
Direct download
3.4.1 (and compiled from source 3.4.2)

Describe the bug / provide steps to reproduce it

Difficult to rep, but the screencap of mached.PNG in a reproducer repo shows that "BIG needles" is flagged as an existence match several times, but on line 15 it is not matched.

You can rep the issue by cloning https://github.com/mikemckiernan/troubleshoot-vale, and then running vale --config ./.vale.ini ./Needle.rst. Line 15 is not included in the response.

I've spent some time trying to narrow this down more. Unfortunately, I'm not familiar with the code or much with Go.

I did step through some of the code in the VS Code debugger and can say that an alert object is created on lint.go:283, but it's somehow discarded by subsequent processing that I haven't noodled through.

@jdkato
Copy link
Member

jdkato commented May 18, 2024

Thanks for the report.

This is caused by the TOC inserting content into the generated HTML that doesn't exist in the source document.

It should be fixed in the next release.

@mikemckiernan
Copy link
Author

Cool, TY! Is there a branch that has the fix?

Closing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants