Skip to content

doc: fix broken using link in ffi.md - #65632

Open
soulee-dev wants to merge 1 commit into
nodejs:mainfrom
soulee-dev:doc-fix-ffi-using-link
Open

doc: fix broken using link in ffi.md#65632
soulee-dev wants to merge 1 commit into
nodejs:mainfrom
soulee-dev:doc-fix-ffi-using-link

Conversation

@soulee-dev

Copy link
Copy Markdown
Contributor

The [using] link in ffi.md points at
proposal-explicit-resource-management/#sec-using-declarations, but that
fragment does not exist, so the link silently resolves to the top of the page.

Point it at the MDN page for using, which fs.md already uses for the same
reference.

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Review requested:

  • @nodejs/ffi

@nodejs-github-bot nodejs-github-bot added doc 问题 and PRs related to Node.js documentation. ffi 问题 and PRs related to experimental Foreign Function Interface support. labels Aug 29, 2026
@daeyeon

daeyeon commented Aug 29, 2026

Copy link
Copy Markdown
Member

Can you rebase onto the latest main? #65636 fixed the CI failures.

The `#sec-using-declarations` fragment does not exist in the explicit
resource management proposal, so the link silently resolves to the top
of the page. The proposal defines `using` under
`#sec-let-and-const-declarations`, and the only `using`-related section
ids are static semantics entries.

Point the reference at the MDN page for `using` instead, matching the
`[`using`]` definition already used in fs.md.

Signed-off-by: Soul Lee <alus20x@gmail.com>
@soulee-dev
soulee-dev force-pushed the doc-fix-ffi-using-link branch from 8249db8 to 6fa9f1a 比较 August 29, 2026 13:40
@soulee-dev

soulee-dev commented Aug 29, 2026

Copy link
Copy Markdown
Contributor Author

@daeyeon Rebased onto the latest main (452c8c5), which includes #65636.
No changes other than the rebase.

Thanks for the heads-up!

注册 for free to join this conversation on GitHub. Already have an account? 登录 to comment

标签

doc 问题 and PRs related to Node.js documentation. ffi 问题 and PRs related to experimental Foreign Function Interface support.

项目

None yet

Development

Successfully merging this pull request may close these issues.

4 participants