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

Lint 규칙상 불필요한 경우에도 dependencies 모듈 사용이 강제되는 문제 해결 #75

Merged
merged 3 commits into from
Nov 6, 2024

Conversation

hoqn
Copy link
Collaborator

@hoqn hoqn commented Nov 6, 2024

✏️ 한 줄 설명

이 PR의 주요 변경 사항이나 구현된 내용을 간략히 설명해 주세요.

모든 파일들에서 사용되는 모듈의 dependencies 사용이 강제되던 문제가 있던 문제를 해결했어요.

✅ 작업 내용

config 파일이나 storybook, test 파일과 같은 dev 환경에서 사용되는 파일에 대해서는 devDependencies를 허용하도록 import/no-extraneous-dependencies 룰을 수정했어요.

🏷️ 관련 이슈

📸 스크린샷/영상

이번 PR에서 변경되거나 추가된 뷰가 있는 경우 이미지나 동작 영상을 첨부해 주세요.

config 파일에 대해서는 다음과 같이 devDependencies를 추가할 수 있습니다.

스크린샷 2024-11-06 오전 10 54 33

나머지 파일에 대해서는 다음과 같이 devDependencies를 사용하면 경고가 발생합니다.

스크린샷 2024-11-06 오전 10 51 25
나머지의 경우 스크린샷 2024-11-06 오후 12 08 50 스크린샷 2024-11-06 오후 12 08 58 스크린샷 2024-11-06 오후 12 10 13 스크린샷 2024-11-06 오후 12 10 19 스크린샷 2024-11-06 오후 12 10 28 스크린샷 2024-11-06 오후 12 10 52

📌 리뷰 진행 시 참고 사항

리뷰 코멘트 작성 시 특정 사실에 대해 짚는 것이 아니라 코드에 대한 의견을 제안할 경우, 강도를 함께 제시해주세요! (1점: 가볍게 참고해봐도 좋을듯 ↔ 5점: 꼭 바꾸는 게 좋을 것 같음!)

@hoqn hoqn requested a review from a team as a code owner November 6, 2024 03:13
@hoqn hoqn requested review from parkblo and heegenie and removed request for a team November 6, 2024 03:13
Copy link
Collaborator

@heegenie heegenie 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
Collaborator

@parkblo parkblo left a comment

Choose a reason for hiding this comment

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

😆

@hoqn hoqn merged commit 9437383 into main Nov 6, 2024
@heegenie heegenie deleted the fix/lint branch November 8, 2024 02:07
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.

Lint 규칙상 불필요한 경우에도 dependencies 모듈 사용이 강제되는 문제 해결
3 participants