[eslint-plugin] 신규 플러그인 / 룰 검토

[yceffort-naver]

O

• eslint-config-standard 제거
  • neostandards 로 대체
• eslint-plugin-yml
  • 현재 yaml은 prettier 에 위임하고 있지만,
  • fomatting -> prettier, 외에 룰 도입

strict config 추가

• eslint-plugin-sonarjs 보류
  • 신규 프로젝트에는 괜찮을듯하지만, 기존 프로젝트에는 파급력이 커보임
  • eslint-plugin-unicorn랑 충돌나는 옵션 고려 필요
• eslint-plugin-unicorn 보류
  • 위와 동일한 이유

---

X

• eslint-plugin-node 대신 eslint-plugin-n 으로 (업데이트가 느려터짐)
  • eslint-plugin-node 제거 됌
• eslint-plugin-workspaces 보류
  • 현재 코드에 변경사항이 크지 않다면 적용해볼만함
  • 워크스페이스 아닌 환경 고려
• eslint-plugin-compat 보류
  • 플러그인은 맘에 들지만 browserlist 기반으로 동작해, 사용하는 쪽에서 별도의 설정이 필요하다는 내용을 명시해줘야함
• eslint-plugin-vitest 보류
  제공자 입장에서 대상 파일을 선언하는 방식은 지양하기 위해 보류

  export default [
      js.configs.recommended,
      {
          files: ['**/__tests__/**', '**/tests/**', '**/*{.,_}{test,spec}.{js,cjs,mjs,jsx,ts,cts,mts,tsx}'],
          ...vitest.configs.recommended,
      },
  ]

• eslint-plugin-css-modules
  • Project status: NOT MAINTAINED
• eslint-plugin-deprecation
  -> deprecated // https://typescript-eslint.io/rules/no-deprecated
  n/no-deprecated-api 가 역할을 하고 있는 것으로 보임

[yceffort-naver]

@NaverPayDev/frontend 혹시 이외에 평소에 쓰고 싶으셨던 룰/플러그인 있으시면 공유 부탁드립니다~

[yongholeeme]

• eslint --print-config file.js 으로 v1 -> v2 변경사항 정리

[yongholeeme]

@byhhh2 님, 저 질문있습니다. eslint-config 에서 테스트 코드 작성했던 기준은 어떤건지 기억하시나요? https://github.com/NaverPayDev/code-style/tree/main/packages/eslint-config/tests

[byhhh2]

@yongholeeme 님~
config를 extends 한 rule 빼고 다 작성했어요! (아마도)

[yongholeeme]

아하, 이해했습니다~~ 참고하겠습니다

[yceffort-naver]

룡호님

https://github.com/sindresorhus/eslint-plugin-unicorn

이것도한번..!

[yongholeeme]

@yceffort-naver 님, 좀 파격적인 플러그인인데요ㅋㅋ

• https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/no-array-for-each.md
• https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/no-array-reduce.md
• https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/no-lonely-if.md
• ...

[yceffort-naver]

몇개만.. 할까요..? 이거 싸다구 맞으려나? ㅋㅋㅋ
사실 제가 완전 실무자도 아닌데 하자고 하기 좀 거시기하기도하네요...

[2-one-week]

https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/no-array-for-each.md

찬성!

갠적으로 중간에 break 못하는 forEach 보단 for-of 선호 ㅎㅎ

https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/no-array-reduce.md
https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/no-lonely-if.md

reduce랑 if는 좀ㅋㅋ

[yceffort-naver]

(내용은 얼추 다 동의하는데..)