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

eslint를 마이너 업그레이드하고 추천 규칙 모음을 사용합니다. #113

Merged
merged 6 commits into from
Aug 27, 2021

Conversation

giwan-dev
Copy link
Contributor

eslint 마이너 버전을 최신으로 업그레이드합니다. 업그레이드하면서 추가된 규칙 설정을 스냅샷에 반영합니다. minor 버전이 오른 것이므로 Breaking Change가 아닙니다.

eslint의 추천 규칙 모음(eslint:recommended)을 사용합니다. 이에 추가된 규칙은 "off"합니다. 따라서, Breaking Change가 아닙니다.

소스코드로 관리하던 규칙 중에 추천 규칙과 겹쳐서 제거해도 괜찮은 규칙을 모두 제거합니다. 그리고 남은 규칙은 rules/base에 정리합니다.

eslint의 major 버전은 같기 때문에 하위호환이 유지됩니다.
추천 규칙 모음을 사용하면서 추가된 규칙은 off 상태로 추가합니다.
eslint의 추천 규칙 모음과 어긋나는 규칙은 자체 규칙이라고 보고,
rules/base로 옮깁니다.
그리고 필요 없어진 recommends/eslint는 제거합니다.
@giwan-dev giwan-dev requested a review from a team as a code owner August 26, 2021 08:51
Copy link
Contributor

@inbeom inbeom left a comment

Choose a reason for hiding this comment

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

👍

@inbeom inbeom requested a review from a team August 27, 2021 00:59
@giwan-dev giwan-dev merged commit 5e44393 into master Aug 27, 2021
@giwan-dev giwan-dev deleted the feature/eslint branch August 27, 2021 01:04
@giwan-dev giwan-dev added this to the v3.0.0 milestone Aug 27, 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.

3 participants