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

⚙️ 프로젝트 추가 설정 #5

Merged
merged 17 commits into from
Oct 13, 2024
Merged

Conversation

Legitgoons
Copy link
Member

@Legitgoons Legitgoons commented Oct 12, 2024

작업 이유

  • 프로젝트 추가 설정

작업 사항

1️⃣ eslint 추가 설정

  • import order, fsd/import, @tanstack/query 3가지 패키지를 추가로 설치, 설정했습니다.

2️⃣ tanstack-query 설정

  • QueryProvider.tsx 파일에 설정했습니다.
  • react query devtools도 함께 설정했습니다.

3️⃣ tailwind css 설정

font 설정

  • woff2 subset font를 기본 font로 설정하고, woff subset font를 fallback font로 사용했습니다.
  • Figma에 정의된 대로 font를 선언해서 사용할 수 있도록 했습니다.
    • text + Figma에 정의된 font name + font size의 첫 글자를 입력해서 사용할 수 있습니다.
    <div className="text-h1b">Hello World</div>;

image

const createFontSize = (
  size: string,
  fontWeight: string,
  lineHeight: string = '130%',
): [string, { letterSpacing: string; lineHeight: string; fontWeight: string | number }] => [
  size,
  { letterSpacing: '-0.03em', lineHeight, fontWeight },
];
//...
fontSize: {
  h1b: createFontSize('24px', 'bold'),
  h1s: createFontSize('24px', '600'),
  h2s: createFontSize('20px', '600'),
  h3b: createFontSize('16px', 'bold', '140%'),
  h3s: createFontSize('16px', '600', '140%'),
  h4s: createFontSize('14px', '600'),
  h4m: createFontSize('14px', '500'),
  b1s: createFontSize('12px', '600'),
  b1m: createFontSize('12px', '500'),
  b1r: createFontSize('12px', '400', '140%'),
  b2s: createFontSize('11px', '600'),
  b2m: createFontSize('11px', '500'),
  b2r: createFontSize('11px', '400'),
  b3m: createFontSize('9px', '500'),
  b4m: createFontSize('8px', '500'),
},

리뷰어가 중점적으로 확인해야 하는 부분

  • �Eslint가 정상적으로 작동하는지
  • query devtools가 정상적으로 작동하는지
  • font가 정상적으로 적용되는지

발견한 이슈

  • 현재 Local 개발 환경에서 eslint가 정상적으로 작동하지 않는 이슈가 있습니다. 혹시 다른 분들도 같은 이슈가 발생하신다면 review에 남겨주시면 감사드리겠습니다.
    • 병준님 덕분에 무사히 해결했습니다😉

@Legitgoons Legitgoons self-assigned this Oct 12, 2024
Copy link
Collaborator

@BangDori BangDori left a comment

Choose a reason for hiding this comment

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

eslint 설정하신다구 고생하셨습니다! 리뷰 확인 부탁드려요~

src/app/provider/QueryProvider.tsx Show resolved Hide resolved
tailwind.config.ts Outdated Show resolved Hide resolved
.eslintrc.json Show resolved Hide resolved
Copy link
Collaborator

@suhwan2004 suhwan2004 left a comment

Choose a reason for hiding this comment

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

eslint, msw 이슈때문에 고생 많습니다... 든든합니다! 간단하게 리뷰 남겨두었어요

tailwind.config.ts Show resolved Hide resolved
eslint.config.mjs와 .eslintrc.json의 충돌로 인해 eslint가 정상적으로 작동하지 않아 삭제했습니다.
React와 Next import문은 반드시 맨 위로 가도록 pathGroupsExcludedImportTypes에 추가했습니다.
사용하지 않는 styled component는 제거했습니다.
Copy link
Collaborator

@BangDori BangDori left a comment

Choose a reason for hiding this comment

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

고생하셨습니당~ eslint alias 설정만 확인 부탁드려요!

tailwind.config.ts Show resolved Hide resolved
.eslintrc.json Outdated Show resolved Hide resolved
@Legitgoons Legitgoons closed this Oct 13, 2024
@Legitgoons Legitgoons reopened this Oct 13, 2024
@BangDori BangDori closed this Oct 13, 2024
@BangDori BangDori reopened this Oct 13, 2024
@Legitgoons Legitgoons merged commit 78da995 into main Oct 13, 2024
2 of 4 checks passed
@Legitgoons Legitgoons deleted the feat/project-setting-2 branch October 13, 2024 09:25
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