Skip to content

모든 컴포넌트 ref 전달 지원#951

Merged
SimYunSup merged 1 commit intomainfrom
842-forward-ref
Apr 17, 2026
Merged

모든 컴포넌트 ref 전달 지원#951
SimYunSup merged 1 commit intomainfrom
842-forward-ref

Conversation

@SimYunSup
Copy link
Copy Markdown
Member

@SimYunSup SimYunSup commented Apr 7, 2026

Closes #842

모든 컴포넌트가 ref props 형식을 사용하여 ref를 전달하도록 변경했습니다(React 19+ 호환 지원).

CI install-test에서 symlink 대신 bun pack tarball 설치 방식으로 변경하여 @types/react 버전 충돌(18 vs 19)을 해결했습니다.

체크 리스트

  • 코드 리뷰를 요청하기 전에 반드시 CI가 통과하는지 확인해주세요.
  • 컴포넌트나 스토리 변경이 있는 경우 PR에 ui 태그를 달아주세요.
    • UI Tests를 통해 스냅샷 차이가 의도된 것인지 확인해주세요.
    • UI Review를 통해 디자이너에게 리뷰를 요청하고 승인을 받으세요.

@SimYunSup SimYunSup requested a review from a team as a code owner April 7, 2026 13:41
@DaleSeo
Copy link
Copy Markdown
Member

DaleSeo commented Apr 8, 2026

@SimYunSup 님, 리뷰 요청 전에 CI에서 실패하는 체크가 없는지 확인 부탁드려요!
2026-04-07 at 21 21 22

@DaleSeo
Copy link
Copy Markdown
Member

DaleSeo commented Apr 8, 2026

본 PR이 어떤 티켓을 위한 건지 연결 부탁드리겠습니다. (본문에 Closes 123 식으로 쓰셔도 됩니다.)
2026-04-07 at 21 22 07

@SimYunSup
Copy link
Copy Markdown
Member Author

@DaleSeo 어떤 ref를 반환해야 하는지 테스트는 불필요한 테스트라고 생각해서 추가하지 않았는데 에러가 뜨네요. 추가해야 할까요?

@DaleSeo
Copy link
Copy Markdown
Member

DaleSeo commented Apr 8, 2026

@DaleSeo 어떤 ref를 반환해야 하는지 테스트는 불필요한 테스트라고 생각해서 추가하지 않았는데 에러가 뜨네요. 추가해야 할까요?

@hyoseong1994 님, Codecov의 patch 타켓 설정이 너무 빡시게 되어 있는 건 아닐까요?

@SimYunSup 그런데 티켓에는 Heading에 ref 전달을 지원하라고 되어 있고, 실제 구현에는 모든 컴포넌트에 적용이 되었네요? 두 분 따로 협의를 하신 건가요?

@hyoseong1994
Copy link
Copy Markdown
Contributor

@hyoseong1994 님, Codecov의 patch 타켓 설정이 너무 빡시게 되어 있는 건 아닐까요?

넵 맞습니다. 엄격한 편입니다. 패치는 기존 대비 1% 이하로만 허용되게 설정했었습니다.
3%로 조정할까요?
다른분들 의견도 듣고싶네요 개인적으로 엄격하다보니 테스트코드 누락 등이 자연스럽게 신경써져서 좋았습니다.

@SimYunSup 그런데 티켓에는 Heading에 ref 전달을 지원하라고 되어 있고, 실제 구현에는 모든 컴포넌트에 적용이 되었네요? 두 분 따로 협의를 하신 건가요?

따로 합의한 내용은 없는것으로 알고있습니다.

@DaleSeo
Copy link
Copy Markdown
Member

DaleSeo commented Apr 8, 2026

@hyoseong1994 3%로 조정 👍

@hyoseong1994
Copy link
Copy Markdown
Contributor

hyoseong1994 commented Apr 8, 2026

@SimYunSup 윤섭님 codecov 조정하였으니 rebase 받으셔서 작업 진행해주시면 감사하겠습니다.

@DaleSeo
Copy link
Copy Markdown
Member

DaleSeo commented Apr 10, 2026

따로 합의한 내용은 없는것으로 알고있습니다.

제가 승인을 해야하는 건지 어떻게 해야하는 건지 잘 모르겠습니다 😅 소통 좀 해주세용 두 분! @SimYunSup @hyoseong1994

Copy link
Copy Markdown
Contributor

@hyoseong1994 hyoseong1994 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@SimYunSup 작업하신 모든 컴포넌트 ref 지원하는것에는 동의합니다.
다만 forwardRef 사용에 대해서는 확인부탁드립니다.

Comment thread src/components/Box/Box.tsx Outdated
Comment thread src/components/Box/Box.tsx Outdated
Drop React 18 peerDependency and replace forwardRef with native
React 19 ref-as-prop pattern across all components. Also update
install-test to use npm pack and vite@latest.
Copy link
Copy Markdown
Contributor

@hyoseong1994 hyoseong1994 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

이미 승인했었는데 다시 요청안하셔도됩니다. ㅎㅎ

@SimYunSup SimYunSup added this pull request to the merge queue Apr 17, 2026
Merged via the queue into main with commit 58e4937 Apr 17, 2026
9 checks passed
@SimYunSup SimYunSup deleted the 842-forward-ref branch April 17, 2026 19:00
@github-actions github-actions Bot mentioned this pull request Apr 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

모든 컴포넌트 ref 전달 지원

3 participants