Skip to content
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.

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

react 17을 준비합니다. #1469

Merged
merged 11 commits into from
Jul 22, 2021
Merged

react 17을 준비합니다. #1469

merged 11 commits into from
Jul 22, 2021

Conversation

giwan-dev
Copy link
Contributor

@giwan-dev giwan-dev commented Jun 29, 2021

설명

react, react-dom, react-test-renderer를 17로 업그레이드하고, react를 peer 의존성으로 가지는 패키지를 react 17을 지원하는 버전으로 업그레이드합니다.

변경 내역 및 배경

storybook에서 react@16 peer-dep 에러가 발생했지만, --legacy-per-deps 옵션으로 우회합니다. storybook author가 권장하는 방법이라;; 울며 겨자먹기로 사용합니다.

사용 및 테스트 방법

canary

이 PR의 유형

🤔

@giwan-dev giwan-dev requested a review from a team as a code owner June 29, 2021 07:49
@giwan-dev
Copy link
Contributor Author

canary release

@github-actions
Copy link

@zprime0920 I got it! 🚀

rhiokim
rhiokim previously approved these changes Jun 29, 2021
Copy link
Contributor

@rhiokim rhiokim left a comment

Choose a reason for hiding this comment

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

👍🏻

@github-actions
Copy link

v2.28.2-pr-1469.22 has been published!

@github-actions github-actions bot temporarily deployed to June 29, 2021 08:00 Inactive
@giwan-dev giwan-dev changed the title react, react-dom, react-test-renderer를 17로 올립니다. react 17을 준비합니다. Jun 30, 2021
@giwan-dev
Copy link
Contributor Author

canary release

@github-actions
Copy link

@zprime0920 I got it! 🚀

@github-actions
Copy link

v2.28.2-pr-1469.24 has been published!

@github-actions github-actions bot temporarily deployed to June 30, 2021 01:04 Inactive
@giwan-dev
Copy link
Contributor Author

canary release

@github-actions
Copy link

@zprime0920 I got it! 🚀

@github-actions
Copy link

v2.28.2-pr-1469.27 has been published!

@github-actions github-actions bot temporarily deployed to June 30, 2021 01:42 Inactive
@inbeom
Copy link
Contributor

inbeom commented Jul 1, 2021

researchgate/react-intersection-observer#205 하나 열고 왔습니다 .. 반응 보시죠

@kooinsung
Copy link
Contributor

kooinsung commented Jul 1, 2021

researchgate/react-intersection-observer#205 하나 열고 왔습니다 .. 반응 보시죠

nibblebot 가 이욜에게 커피한잔 기부해야겠네여 ㅋㅋ

@giwan-dev
Copy link
Contributor Author

https://github.com/researchgate/react-intersection-observer/pull/196/files
PR 목록 읽다가 이런 걸 봐버렸습니다... 레포가 버려진 걸까요ㅠㅠ

@inbeom
Copy link
Contributor

inbeom commented Jul 1, 2021

ㅋㅋㅋ 저도 봤는데 봇이라 모르시는 것 같아갖구...

이번 주까지 반응 없으면 포크 해버리시죠

@giwan-dev giwan-dev marked this pull request as draft July 2, 2021 06:30
@inbeom
Copy link
Contributor

inbeom commented Jul 5, 2021

포크하려고 들어왔는데 메인테이너가 이모지를 남겨두셨네요 researchgate/react-intersection-observer#205 뭐지..

@inbeom
Copy link
Contributor

inbeom commented Jul 20, 2021

https://www.npmjs.com/package/@titicaca/react-intersection-observer 1.5.0으로 릴리즈 해뒀습니다 .. npm 으로는 잘 안 되고 Yarn 필수군요.

@giwan-dev
Copy link
Contributor Author

@inbeom 배포할 때 그냥 yarn run release 하면 되나요? prop 타입이 export 안 되어있어서 export하려고 합니당

@inbeom
Copy link
Contributor

inbeom commented Jul 20, 2021

yarn publish 하면 됩니다용!

@giwan-dev giwan-dev marked this pull request as ready for review July 20, 2021 05:15
@giwan-dev giwan-dev added this to the v2.31.0 milestone Jul 20, 2021
@giwan-dev giwan-dev requested a review from a team July 20, 2021 05:16
inbeom
inbeom previously approved these changes Jul 21, 2021
Copy link
Contributor

@inbeom inbeom left a comment

Choose a reason for hiding this comment

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

👏

@inbeom inbeom requested a review from a team July 21, 2021 00:19
그냥 설치하면 storybook의 의존성에서 에러가 발생해서 `--legacy-peer-deps` 옵션을 사용합니다.
storybook author의 답변입니다;;
@types/googlemaps 설치할 때 @types/google.maps를 사용하라는 deprecate 경고 메시지가 뜹니다.
하지만 @react-google-maps/api가 여전히 이 타입 정의를 사용하고 있기 때문에
@types/google.maps를 사용하지 않습니다.
최신 버전에서 react 17을 peer dependency로 가지기 때문에 업그레이드 합니다.
react 17을 지원하는 버전을 설치합니다.
`@researchgate/react-intersection-observer`를 대체합니다.
@giwan-dev
Copy link
Contributor Author

canary release

@github-actions
Copy link

@zprime0920 I got it! 🚀

@github-actions
Copy link

v2.30.3-pr-1469.41 has been published!

@github-actions github-actions bot temporarily deployed to July 22, 2021 01:23 Inactive
@giwan-dev giwan-dev requested review from inbeom and a team and removed request for a team July 22, 2021 01:49
Copy link
Contributor

@inbeom inbeom left a comment

Choose a reason for hiding this comment

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

🚀

@inbeom inbeom requested a review from a team July 22, 2021 02:14
Copy link
Contributor

@torres-triple torres-triple left a comment

Choose a reason for hiding this comment

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

🚀

@giwan-dev giwan-dev merged commit 019e993 into main Jul 22, 2021
@giwan-dev giwan-dev deleted the update-react-17 branch July 22, 2021 02:53
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.

range-picker 의존 모듈이 업데이트되면서 발생하는 type 이슈를 개선합니다.
5 participants