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

refactor(@yourssu/eslint-config): remove rushstack-patch #54

Merged
merged 5 commits into from
Jun 28, 2024

Conversation

intersoom
Copy link
Member

1️⃣ 어떤 작업을 했나요? (Summary)

기존 코드에 영향을 미치는 변경사항

스크린샷 2024-06-26 오후 10 13 54

patch 파일이 rushstack 내부 dependecy로 존재해서 설치가 불필요하더라고요 !
사용처에서 따로 require를 해주지 않아도 rushstack 내부에 존재하는 것만으로도 플러그인을 모두 설치해주어서 잘 작동합니다.

일단 저희는 rushstack을 사용하니까 patch 관련 파일은 제거하도록 할게요

2️⃣ 알아두시면 좋아요!

대신 rushstack에 대한 의존성을 제거할 경우에는 별도로 patch를 import 해주어야 할 것 같습니다~
(rushstack/eslint-config에 대한 의존성까지 제거한 경우는 이렇게 되더라고요!)
스크린샷 2024-06-27 오전 3 51 53

3️⃣ 체크리스트 (Checklist)

  • main 브랜치의 최신 코드를 pull 받았나요?

@intersoom intersoom added refactor eslint-config @yourssu/eslint-config labels Jun 26, 2024
@intersoom intersoom self-assigned this Jun 26, 2024
Copy link
Member

@Hanna922 Hanna922 left a comment

Choose a reason for hiding this comment

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

수고하셨습니다! 사용 방법이 달라짐에 따라 문서도 업데이트 되어야 할 것 같아요
import 관련 내용도 포함 부탁드려용

@intersoom intersoom requested a review from Hanna922 June 27, 2024 17:43
@intersoom
Copy link
Member Author

수고하셨습니다! 사용 방법이 달라짐에 따라 문서도 업데이트 되어야 할 것 같아요 import 관련 내용도 포함 부탁드려용

일단 현 버전의 저희 @yourssu/eslint-config를 사용하는 경우에는 import를 해야하는 경우가 없어서
patch.js 파일을 아예 제거를 해버렸습니다!
그래서 만약 저희 팀에서 아예 rushstack 사용을 안 하는 경우에 추가하도록 할게요!

@Hanna922 Hanna922 changed the title refactor: remove rushstack-patch refactor(@yourssu/eslint-config): remove rushstack-patch Jun 28, 2024
Copy link
Member

@Hanna922 Hanna922 left a comment

Choose a reason for hiding this comment

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

👍🏻

@intersoom intersoom merged commit f3dc094 into main Jun 28, 2024
@intersoom intersoom deleted the refactor/#42-eslintPatch branch June 28, 2024 11:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
eslint-config @yourssu/eslint-config refactor
Projects
None yet
Development

Successfully merging this pull request may close these issues.

refactor: @rushstack/eslint-patch 필요성 확인
2 participants