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

전남대 FE_이도현_1주차 과제 Step1 #54

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

leedyun
Copy link

@leedyun leedyun commented Jun 28, 2024

안녕하세요 멘토님. 전반적으로 맞게 했는지 궁금하지만 가장 궁금한 부분을 다음과 같이 문의드립니다.

  1. Lint룰 세팅을 어떻게 하면 좋을지 한 가지만 예시를 들어주실 수 있을까요?
  2. 불필요한 파일이나 코드 삭제가 올바르게 됐는지 궁금합니다.

"@testing-library/jest-dom": "^5.17.0",
"@testing-library/react": "^13.4.0",
"@testing-library/user-event": "^13.5.0",
"@types/jest": "^27.5.2",
Copy link

Choose a reason for hiding this comment

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

일반적으로 @types/* dependency 는 devDependencies에 정의되는게 좋습니다. 그 이유는

  • 개발 중에만 필요: 타입 정의 파일은 TypeScript 컴파일러가 코드 검사를 수행할 때만 필요합니다. 실제 애플리케이션 실행 시에는 전혀 사용되지 않습니다.

  • 프로덕션 빌드 크기 최소화: 타입 정의 파일을 devDependencies에 넣으면 프로덕션 환경에 배포할 때 불필요한 파일을 포함하지 않게 되어 애플리케이션 크기를 줄일 수 있습니다.

@tlsehdfl
Copy link

tlsehdfl commented Jul 3, 2024

Q ) Lint룰 세팅을 어떻게 하면 좋을지 한 가지만 예시를 들어주실 수 있을까요?

처음부터 일일이 설정하려면 어려우실 수 있습니다.

npx eslint --init

로 초기화 한 이후에 필요한 설정만 변경 & 추가 하는 방식을 추천드려요.

Q ) 불필요한 파일이나 코드 삭제가 올바르게 됐는지 궁금합니다.
A ) logo.svg 를 제거하고 App.css, index.css 의 불필요한 스타일을 제거하면 좋을거 같습니다.

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