Skip to content
Merged
Changes from all commits
Commits
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
49 changes: 49 additions & 0 deletions contributions/65632.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,49 @@
---
pr-url: https://github.com/nodejs/node/pull/65632
---

## 문제

`doc/api/ffi.md` 문서 하단 `[`using`]`에서 존재하지 않는 링크(fragment)를 가리키고 있었습니다.

```md
[`using`]: https://tc39.es/proposal-explicit-resource-management/#sec-using-declarations
```

`#sec-using-declarations` 링크(fragment)가 해당 페이지에는 없었고, 결국 해당 문서 페이지 최상단으로 넘어가는 ㅏ상태였습니다.

추가적으로 같은 `using`을 참조하는 `doc/api/fs.md`는 이미 MDN 문서를 가리키고 있었습니다.


## 해결

`ffi.md`의 링크를 `fs.md`가 쓰던 MDN 페이지로 맞췄습니다.

```md
[`using`]: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Statements/using
```

## 검증

리뷰 과정에서 멘토님이님이 GHA 실패가 보이니 최신 main으로 rebase를 요청해 주셨고, main으로 rebase 하였습니다.

이후 Collaborator `@trivikr`님이 "doc 변경은 Jenkins CI를 돌리지 않으니 rebase가 필요 없었을 것"이라는 의견을 남겨주셨지만, 멘토님께서 실패한 테스트가 Jenkins가 아니라 GHA라는 점을 말씀해 주셨습니다.

## 결과

- [PR #65632](https://github.com/nodejs/node/pull/65632)
- 2026년 8월 29일 제출
- 커밋: `6fa9f1a2716eee19f474aad30f62789e2bdecbd3`
- 리뷰어 4명 승인 (`@trivikr`, `@cjihrig`, `@jasnell`, `@lpinca`)
- `author ready`, `commit-queue` 라벨이 붙어 병합 대기 중

## 회고

CI 실패의 원인을 두고 리뷰어들 사이에 짧은 논의가 있었는데, 다행히 GHA도 문제 없이 Pass 되어, Review 완료 되었습니다.

## 참고 자료

- [`doc/api/ffi.md`](https://github.com/nodejs/node/blob/main/doc/api/ffi.md)
- [MDN — `using`](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Statements/using)
- [PR #65636](https://github.com/nodejs/node/pull/65636)
- [contributions/65342.md](./65342.md)