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

chore: pnpm으로 마이그레이션을 진행합니다. #127

Merged
merged 5 commits into from
Sep 13, 2024

Conversation

jungwoo3490
Copy link
Member

변경사항

close #125

yarn berryTurborepo 간 호환성 문제로 인해 yarn berry에서 pnpm으로 마이그레이션했어요.

마이그레이션 후 기존 스크립트 다 정상적으로 동작하는 것을 확인했어요.

.npmrc에 추가한 옵션들은 버전 호환성 및 충돌을 방지하기 위해 추가해준 옵션이에요. 자세한 내용은 제가 타 프로젝트에서 호환성 및 충돌 방지를 해결하기 위해 고민했던 PR을 첨부할테니 해당 내용 참고하면 좋을 것 같아요!!

Setaday/Setaday-Client#8

링크

https://sopt-makers.slack.com/archives/C07L351P3JN/p1725544188920659

시급한 정도

🏃‍♂️ 보통 : 최대한 빠르게 리뷰 부탁드립니다.

Copy link

height bot commented Sep 12, 2024

Link Height tasks by mentioning a task ID in the pull request title or commit messages, or description and comments with the keyword link (e.g. "Link T-123").

💡Tip: You can also use "Close T-X" to automatically close a task when the pull request is merged.

Copy link

changeset-bot bot commented Sep 12, 2024

⚠️ No Changeset found

Latest commit: 03d9e61

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@Brokyeom
Copy link
Member

해당 변경은 minor bump로 올리면 좋을 것 같습니다.

@jungwoo3490
Copy link
Member Author

jungwoo3490 commented Sep 13, 2024

해당 변경은 minor bump로 올리면 좋을 것 같습니다.

@Brokyeom
이전에 테스트 코드 추가한 내용 + pnpm으로 script 변경한 내용 summary에 작성했어요.
ui 패키지를 제외한 다른 패키지 같은 경우 마이그레이션으로 인한 변경사항이 없어 ui 패키지만 변경 내용이 추적되어서 해당 패키지만 minor bump로 올렸습니다.

Copy link
Member

@Brokyeom Brokyeom left a comment

Choose a reason for hiding this comment

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

여러 번경으로 프로젝트를 더 좋은 방향으로 끌고가는 것 같아 좋습니다.
너무 고생 많았어요! 앞으로 mds가 이 틀 안에서 더 많은 결과물을 낼 수 있길 기대합니다.

@Brokyeom Brokyeom merged commit 30fc469 into main Sep 13, 2024
@Brokyeom Brokyeom deleted the chore_#125-pnpm-migration branch September 13, 2024 12:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

chore: pnpm으로 마이그레이션을 진행합니다.
2 participants