fix: @typescript-eslint/naming-convention 이 실제로 잘 동작하도록 eslintrc.js의 설정을 변경합니다. #111
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #111 +/- ##
=======================================
Coverage 99.51% 99.51%
=======================================
Files 13 13
Lines 206 206
Branches 46 46
=======================================
Hits 205 205
Misses 1 1 |
| @@ -8,5 +8,5 @@ | |||
| "strict": true, | |||
| "skipLibCheck": true | |||
| }, | |||
| "include": ["src"] | |||
| "include": ["src", ".eslintrc.js"] | |||
There was a problem hiding this comment.
@okinawaa 요건 저도 987d0fd 를 푸시해보고 ci에서 뱉은 에러를 보고 알게되어 추가하게 되었어요!
.eslintrc.js의 parserOptions.project: true 옵션을 추가하면서 ts의 도움을 받게 되었는데요, 이때 eslint에 의해 .eslintrc.js도 린팅 대상에 포함되었으나 설정에 추가되어 있지 않아 발생하는 에러라고 해요.
그래서 해당 파일도 린팅할지 말지 여부에 따라 설정을 변경하여 해당 에러를 해결할 수 있는데용,
- 린팅에 포함시키지 않으려면
.eslintignore에.eslintrc.js를 포함시키는 방법 - 포함시키려면 아래의 방법들이 있는데요,
tsconfig.json의includes에 포함시키는 방법 (2-1)tsconfig.json을extends한tsconfig.eslint.json을 만들고,.eslintrc.js를 포함시키는 방법 (2-2)
이 중에서 변경도 최소화 하면서, 설정 파일도 같이 린팅에 포함되도록 2-1 방법으로 처리해 주었습니다 🙇 (참고한 스택오버플로우, 공식문서)
There was a problem hiding this comment.
@okinawaa 혹시 요거 머지해도 괜찮을까요~?! 🙏
Overview
docs아래 경로에서
yarn lint를 돌려보았을 때, 아래와 같이 경고메시지가 나타나고 있어 해당 문제를 수정한 PR이에요.project: true옵션을 주게 되면, eslint가 동작하는 각 파일이 위치하는 곳의 가장 가까운 곳의tsconfig.json을 매칭시켜 적용하게 돼요. (문서)parserOptions.parser를 설정하지 않은 경우, naming-convention에 맞지 않는 snake_case의 variable이 통과함parserOptions.parser를 true로 설정한 경우, naming-convention에 맞지 않는 snake_case의 variable에 에러를 띄워줌PR Checklist