Skip to content

Chore: eslint&prettier 설정#29

Open
seojin15 wants to merge 3 commits intodevelopfrom
chore/eslint-prettier-setting/#8
Open

Chore: eslint&prettier 설정#29
seojin15 wants to merge 3 commits intodevelopfrom
chore/eslint-prettier-setting/#8

Conversation

@seojin15
Copy link
Copy Markdown
Collaborator

📌 Summary

📚 Tasks

  • ESLint, Prettier 설정

🔍 Describe

  • ESLint, Prettier 설정 했습니다!
  • 저장 시 자동 포맷을 위해 VS Code 설정도 추가했습니다.

👀 To Reviewer

  • 처음 해본 거라... 잘 한건지 확인 부탁드릴게요..!

@seojin15 seojin15 self-assigned this May 10, 2026
@seojin15 seojin15 added 🪜 Chore 그 외 기타 작업 🐰 서진 WEB 38기 황서진 labels May 10, 2026
@seojin15 seojin15 linked an issue May 10, 2026 that may be closed by this pull request
@seojin15 seojin15 changed the title [Chore] eslint&prettier 설정 Chore: eslint&prettier 설정 May 10, 2026
Copy link
Copy Markdown
Collaborator

@gyeongbibin gyeongbibin left a comment

Choose a reason for hiding this comment

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

처음해보시는 건데 잘 설정해주신 것같아요 ! 몇가지 코멘트 남겨두었으니 확인부탁드릴게요 ! 그리고 리뷰 반영 전에 충돌 한 번 잡아주세요 ! Push하기 전에 디벨롭 브랜치를 pull한 이후에 push하면 충돌이 발생하지 않아요 .

Comment thread eslint.config.js Outdated
import reactHooks from "eslint-plugin-react-hooks";
import reactRefresh from "eslint-plugin-react-refresh";
import tseslint from "typescript-eslint";
import prettierConfig from "eslin-config-prettier";
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

eslin -> eslint 오타 났어요 !

Comment thread eslint.config.js Outdated
import reactRefresh from "eslint-plugin-react-refresh";
import tseslint from "typescript-eslint";
import prettierConfig from "eslin-config-prettier";
import prettierPlugin, { rules } from "eslint-plugin-prettier";
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

rules import는 eslint-plugin-prettier 내부 규칙 객체를 가져오는 건데, rules는 사용하지 않아서
import prettierPlugin from "eslint-plugin-prettier"; 이렇게만 둬도 될거같네용

Comment thread eslint.config.js Outdated
"simple-import-sort": simpleImportSort,
},
rules: {
"prettier/prettier": error,
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

"prettier/prettier": "error" 처럼 error를 문자열로 !

Comment thread eslint.config.js Outdated

export default defineConfig([
globalIgnores(['dist']),
globalIgnores(["dist", "node-modules"]),
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

node-modules → node_modules 오타 수정 부탁드립니다!

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

오타가 많았네요ㅜㅜ 다음부턴 더 꼼꼼하게 확인하겠습니다...!! 감사합니다🙇‍♀️

@jyeon03
Copy link
Copy Markdown
Member

jyeon03 commented May 11, 2026

충돌 해결이랑 추가로 셋팅할 게 좀 있어서 서진이 양해구하고 제가 수정 좀 했습니다!!

수정사항

  • prettier-plugin-tailwindcss를 추가하여 저희가 작성하는 Tailwind 클래스들이 공식 권장 순서에 맞게 자동으로 정렬되도록 했어요!
  • @trivago/prettier-plugin-sort-imports를 통해 상단의 Import 구문들도 (React -> 외부 라이브러리 -> 절대 경로 -> 상대 경로) 순으로 깔끔하게 자동 분류되도록 세팅했습니다.
  • 기존 eslint.config.js에 있던 오타들을 수정하고, Prettier와 기능이 겹쳐 충돌을 생기게 했던 eslint-plugin-simple-import-sort 패키지는 제거했습니다.
  • VSCode 공통 설정(.vscode)을 추가해서 팀원 간의 개발 환경을 하나로 통일했습니다.
    -> 프로젝트를 열면 필요한 확장 프로그램(ESLint, Prettier 등) 설치 안내가 뜨고, settings.json을 통해 파일 저장 시 자동으로 포맷팅과 린트 수정이 적용되도록 강제했습니다!!

변경된 환경에서 Ctrl+s로 테스트도 해봤습니다!!!

Copy link
Copy Markdown
Collaborator

@chungyo chungyo left a comment

Choose a reason for hiding this comment

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

고생하셨어용~ 처음 하시는건데 열심히 하신게 보이네요..

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🐰 서진 WEB 38기 황서진 🪜 Chore 그 외 기타 작업

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Chore] ESLint, Prettier 설정

4 participants