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

Feature/set yarn berry/tmf 27 #185

Merged
merged 12 commits into from
Sep 28, 2021
Merged

Conversation

sunwoo0706
Copy link
Member

개요

프로젝트 규모가 커짐에 따라 효율적인 종속성 관리 툴이 필요하여 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 를 입력하여 적용시켰습니다.

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.

수고하셨습니당 😎

Copy link
Contributor

@ImChangGyu ImChangGyu left a comment

Choose a reason for hiding this comment

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

고생 많으셨습니다.

Copy link
Member

@songsihyeon songsihyeon left a comment

Choose a reason for hiding this comment

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

수고하셨습니다.......

@sunwoo0706 sunwoo0706 merged commit b44edea into develop Sep 28, 2021
@delete-merged-branch delete-merged-branch bot deleted the feature/setYarnBerry/TMF-27 branch September 28, 2021 08:51
@sunwoo0706
Copy link
Member Author

sunwoo0706 commented Sep 28, 2021

unplugged 파일들은 gitignore 시켜줘야 하는데 깜빡했네요..
따로 브랜치 파서 진행하도록 하겠습니다.
#187


@iseolin76 @ImChangGyu @songsihyeon

This was referenced Sep 28, 2021
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.

4 participants