Skip to content

fix(scripts): replace stale line-number citations in js-comment-mask.mjs with content anchors #7930

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

Merged
baozhoutao merged 1 commit into from
Sep 6, 2026
+3 −2
Merged

fix(scripts): replace stale line-number citations in js-comment-mask.mjs with content anchors #7930

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 3 additions & 2 deletions scripts/js-comment-mask.mjs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -93,8 +93,9 @@
* (objectstack#10640): deleting the brace counting inside `${...}` fails a case
* below, while dropping `return` from `REGEX_AFTER_KEYWORD` passes all of them
* and is caught only by the corpus sweep. The shape that mutation misreads is
* live in THIS tree too -- `scripts/check-changeset-presence.mjs:555` and
* `scripts/check-doc-links.mjs:616` both write `return /.../` today -- so in
* live in THIS tree too -- `scripts/check-changeset-presence.mjs`'s
* `isDocumentation` function, and the `return /\.(md|mdx)$/.test(root)` site
* in `scripts/check-doc-links.mjs`, both write `return /.../` today -- so in
* objectui that mutation is undetected by anything that runs. Recorded as a
* known limit of the port, not as coverage.
*
Expand Down
Loading