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

chore: 세팅 내용 일부 변경 #14

Merged
merged 2 commits into from
Jul 6, 2024
Merged

chore: 세팅 내용 일부 변경 #14

merged 2 commits into from
Jul 6, 2024

Conversation

wokbjso
Copy link
Member

@wokbjso wokbjso commented Jul 4, 2024

🤔 어떤 문제가 발생했나요?

  1. pnpm 패키지 매니저와 연관 없는 명령어가 존재 (npx)
  2. eslintrc.json 파일에서 prettier 관련 플러그인을 넣어줄 때, "prettier" 로 "plugins" 와 "extends" 속성에 각각 넣어줌으로 인해, prettier와 eslint를 동시에 사용할 때 발생할 수 있는 내부적인 이슈를 막지 못함
  3. next.config.mjs 파일이 eslint 수행 파일에 포함되어 빨간줄이 뜸
  4. 지영님의 건의 -> 세미콜론 필수로 하자!!

🎉 어떻게 해결했나요?

  1. pnpm 명령어로 통합
  2. "plugin:prettier/recommended" 를 extends에 넣어줌으로써 통합
    -> 이 때 발생하는 Delete eslint 오류를 방지하기 위해 eslintrc.json 파일에 prettier 관련 rules 추가
  3. ignorePatterns 속성에 next.config.mjs 파일 추가
  4. 지영님의 의견 반영 -> 세미콜론 필수입니다 ㅎㅎ

📷 이미지 첨부 (Option)

image

⚠️ 유의할 점! (Option)

Copy link
Member

@Jungjjeong Jungjjeong left a comment

Choose a reason for hiding this comment

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

👍👍👍

@wokbjso wokbjso merged commit cd2d41e into main Jul 6, 2024
1 check passed
@wokbjso wokbjso deleted the chore/setting branch July 6, 2024 05:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore # Chores
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants