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

Set yarn berry #180

Closed
wants to merge 12 commits into from
Closed

Set yarn berry #180

wants to merge 12 commits into from

Conversation

sunwoo0706
Copy link
Member

@sunwoo0706 sunwoo0706 commented Sep 23, 2021

개요

프로젝트 규모가 커짐에 따라 효율적인 종속성 관리 툴이 필요하여 yarn berry 설정을 하였습니다.

작업사항

기본 설정 관련

yarnpkg.com을 보고 플러그 앤 플레이 방식으로 yarn berry 설정을 하였고 그 이후 발생된 이슈들을 처리했습니다. 마이그레이션시 참고한 자료입니다. 📖 <

.pnp.cjs 에 플러그 앤 플레이가 디폴트로 설정되어 있기는 하지만 명시해주면 좋을것 같아 아래 코드를 삽입했습니다.

nodeLinker: "pnp"

그리고 플러그 앤 플레이 방식으로 변경됨에 따라 기존의 node-modules (gitignore당해서 github에는 존재하지 않음)와 package-lock.json을 제거했습니다.

그리고 package.json에서는 eslint 설정을 불러올 수 없기에 .eslintrc.json에 다음과 같은 코드를 옮겼습니다.

{
    "extends": [
        "react-app",
        "react-app/jest"
    ]
}

yarn script 관련

yarn start 실행시 발생하는 에러들이었습니다.

  1. command not found
    react-script start 앞에 export 8080 ... 코드때문에 실행이 되지 않았습니다.
  2. react-script 안됨
    yarn add eslint-config-react-app 을 입력하여 해결했습니다. 관련 자료
  3. react-router not declare
    yarn add react-router eslint-config-react-app 을 입력하여 해결하였습니다. 관련 자료
  4. styled-component 로 인해 실행 안됨
    yarn add react-is 를 입력하여 해결했습니다. 관련 자료

type-script 관련

type-script 적용하기 위해서 yarn dlx @yarnpkg/sdks vscode 를 입력하여 적용시켰습니다.

@sunwoo0706 sunwoo0706 changed the title Set yarn to version berry Set yarn berry Sep 23, 2021
@sunwoo0706 sunwoo0706 self-assigned this Sep 23, 2021
Copy link
Contributor

@iseolin76 iseolin76 left a comment

Choose a reason for hiding this comment

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

수고하셨습니다

.eslintrc.json Show resolved Hide resolved
@iseolin76 iseolin76 self-requested a review September 23, 2021 14:33
Copy link
Contributor

@iseolin76 iseolin76 left a comment

Choose a reason for hiding this comment

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

LGTM

@sunwoo0706 sunwoo0706 removed their assignment Sep 23, 2021
@sunwoo0706
Copy link
Member Author

이제 리뷰해주시면 됩니다 !!
@iseolin76 @songsihyeon @ImChangGyu

@sunwoo0706
Copy link
Member Author

sunwoo0706 commented Sep 28, 2021

conflict로 인하여 develop에 있는 코드로 세팅하도록 하겠습니다.
#185

@sunwoo0706 sunwoo0706 closed this Sep 28, 2021
@delete-merged-branch delete-merged-branch bot deleted the feature/setYarnBerry branch September 28, 2021 07:37
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.

2 participants