Skip to content

Conversation

@EATSTEAK
Copy link

1️⃣ 어떤 작업을 했나요? (Summary)

  • react/jsx-runtime을 번들링하지 않도록 하여 react@19(및 가까운 미래 버전)에서 라이브러리가 동작하도록 하였습니다.
  • 추가로 불필요한 UMD variable 제거했습니다.
  • 번들링 사이즈가 줄어 성능 개선 효과도 있을 것으로 예상

기존 코드에 영향을 미치지 않는 변경사항

기존 코드에 영향을 미치는 변경사항

버그 픽스

2️⃣ 알아두시면 좋아요!

3️⃣ 추후 작업

4️⃣ 체크리스트 (Checklist)

  • main 브랜치의 최신 코드를 pull 받았나요?

@EATSTEAK EATSTEAK added the bug Something isn't working label Feb 23, 2025
@EATSTEAK EATSTEAK self-assigned this Feb 23, 2025
Copy link
Collaborator

@nijuy nijuy left a comment

Choose a reason for hiding this comment

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

당신천재야~~~

저 궁금한 게 있는데, 지금은 package.json에 peerDependencies가 이렇게 되어있거든요

  "peerDependencies": {
    "react": "^18.2.0",
    ...
  }

이번 PR에서 19도 지원이 되게끔 수정했으니 여기도 같이 수정해야 하진 않을까요??
그대로 두면 react 19 환경에서 설치했을 때 경고 같은 게 뜨진 않을까 해서요

(직접 확인해보고 코멘트 달려고 했는데 나가야 해서 일단 어푸룹 먼저 합니다 지송 ^ ^ ....)

@EATSTEAK
Copy link
Author

당신천재야~~~

저 궁금한 게 있는데, 지금은 package.json에 peerDependencies가 이렇게 되어있거든요


  "peerDependencies": {

    "react": "^18.2.0",

    ...

  }

이번 PR에서 19도 지원이 되게끔 수정했으니 여기도 같이 수정해야 하진 않을까요??

그대로 두면 react 19 환경에서 설치했을 때 경고 같은 게 뜨진 않을까 해서요

(직접 확인해보고 코멘트 달려고 했는데 나가야 해서 일단 어푸룹 먼저 합니다 지송 ^ ^ ....)

보통

"peerDependencies": {
  "@types/react": ">=17.0.0",
  "react": ">=17.0.0"
},

이런식으로 적는데 적절히 range 설정하면 될 것 같아요.

그런데 19 동작이 아직 완전 보장된 건 아니니까(아이콘만 확인함) 확인해보고 변경해도 좋지 않을까 싶네요.

이미 deprecated이니까 오류메시지를 띄우는게 맞지 않나 싶기도 하고요

@EATSTEAK EATSTEAK merged commit 182dd7e into develop Feb 26, 2025
1 check passed
@fecapark
Copy link
Collaborator

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants