Skip to content

Respect TeX control boundaries and verbatim ranges - #305

Merged
1sgtpepper merged 14 commits into
mainfrom
fix/latex-control-and-verbatim
Aug 11, 2026
Merged

Respect TeX control boundaries and verbatim ranges#305
1sgtpepper merged 14 commits into
mainfrom
fix/latex-control-and-verbatim

Conversation

@1sgtpepper

@1sgtpepper 1sgtpepper commented Aug 4, 2026

Copy link
Copy Markdown
Owner

Summary

LaTeX scanning now honors active TeX control boundaries and source-ordered ownership of
comments and verbatim regions. Controls inside comments or verbatim text remain opaque;
live controls, newline and EOF comments, matched closers, and mismatched verbatim
closers retain their required behavior. Markdown math labels use the same active-control
boundary rule.

Linked issues

Fixes #264.
Fixes #268.
Fixes #269.

Validation

Exact-head GitHub Actions passed on 8b161799bfbd0f12dd0a12215c0756183c4c14d0:
CI run 31526368893 and
Docs run 31526368929.
quality, docs, package, public regression replay, self-check, Python 3.11–3.13,
aggregate test, build-docs, and codecov/patch all passed.

@1sgtpepper
1sgtpepper force-pushed the fix/latex-control-and-verbatim branch from 577fe16 to 03bcd0e Compare August 6, 2026 21:02
@codecov

codecov Bot commented Aug 7, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 98.97959% with 1 line in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/scieqlint/scan/latex.py 98.92% 0 Missing and 1 partial ⚠️

📢 Thoughts on this report? Let us know!

@1sgtpepper 1sgtpepper added bug Something isn't working area:parser Grammar, AST, and unsupported syntax behavior. area:scanner-latex LaTeX source scanning. needs design Not ready for implementation. needs fixture Needs example input/output coverage. blocked Waiting on another issue or release. labels Aug 7, 2026
@1sgtpepper
1sgtpepper force-pushed the fix/latex-control-and-verbatim branch 3 times, most recently from 8c73d79 to 39a5ca1 Compare August 9, 2026 05:43
@1sgtpepper 1sgtpepper removed the needs fixture Needs example input/output coverage. label Aug 10, 2026
@1sgtpepper
1sgtpepper changed the base branch from main to fix/dollar-math-boundaries August 10, 2026 17:35
@1sgtpepper
1sgtpepper force-pushed the fix/latex-control-and-verbatim branch 4 times, most recently from f2dce70 to 3ce5ee2 Compare August 10, 2026 19:29
@1sgtpepper 1sgtpepper removed needs design Not ready for implementation. blocked Waiting on another issue or release. labels Aug 10, 2026
@1sgtpepper
1sgtpepper marked this pull request as ready for review August 10, 2026 19:35
@1sgtpepper
1sgtpepper force-pushed the fix/latex-control-and-verbatim branch 3 times, most recently from 844ca7d to cb3d8ea Compare August 11, 2026 13:18
@1sgtpepper
1sgtpepper force-pushed the fix/dollar-math-boundaries branch from bd20027 to a9c59f8 Compare August 11, 2026 15:39
@1sgtpepper
1sgtpepper force-pushed the fix/latex-control-and-verbatim branch 4 times, most recently from b57c035 to 474fd48 Compare August 11, 2026 17:39
@1sgtpepper
1sgtpepper force-pushed the fix/latex-control-and-verbatim branch from 474fd48 to 8b16179 Compare August 11, 2026 19:08
@1sgtpepper
1sgtpepper changed the base branch from fix/dollar-math-boundaries to main August 11, 2026 19:08
@1sgtpepper
1sgtpepper merged commit db28a54 into main Aug 11, 2026
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:parser Grammar, AST, and unsupported syntax behavior. area:scanner-latex LaTeX source scanning. bug Something isn't working

Projects

None yet

1 participant