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

refactor: 프로젝트 구조 변경 #101

Merged
merged 13 commits into from
Dec 12, 2024
Merged

refactor: 프로젝트 구조 변경 #101

merged 13 commits into from
Dec 12, 2024

Conversation

gwansikk
Copy link
Member

@gwansikk gwansikk commented Dec 12, 2024

Summary

점직전으로 프로젝트의 유지보수성을 업데이트 합니다. 해당 PR은 유지보수를 더욱 편하도록 개선하고자 1차적으로 적용했습니다.

Tasks

  • 프로젝트 구조를 더욱 간소화 했습니다. (필요한 함수는 가까운 곳에, FSD와 비슷하게)
  • 일관된 코드 컨벤션을 적용했습니다.
  • Yarn Berry를 업데이트 했습니다.
  • 서버와 동일한 스키마 타입을 더욱 단순하게 재정의했습니다.
  • 모든 컴포넌트와 코드를 최적화 및 단순화 했습니다.
  • SplashPage를 비활성화 했습니다.

@gwansikk gwansikk self-assigned this Dec 12, 2024
@gwansikk gwansikk requested a review from Copilot December 12, 2024 10:39

Choose a reason for hiding this comment

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

Copilot reviewed 74 out of 88 changed files in this pull request and generated no suggestions.

Files not reviewed (14)
  • .eslintcache: Language not supported
  • .prettierignore: Language not supported
  • .vscode/settings.json: Language not supported
  • .yarn/sdks/prettier/package.json: Language not supported
  • .yarn/sdks/typescript/package.json: Language not supported
  • .yarnrc.yml: Language not supported
  • index.html: Language not supported
  • package.json: Language not supported
  • .github/ISSUE_TEMPLATE/bug.yml: Evaluated as low risk
  • .yarn/sdks/typescript/lib/tsserverlibrary.js: Evaluated as low risk
  • .yarn/sdks/typescript/lib/tsserver.js: Evaluated as low risk
  • .yarn/sdks/typescript/lib/tsc.js: Evaluated as low risk
  • .yarn/sdks/eslint/lib/unsupported-api.js: Evaluated as low risk
  • .yarn/sdks/typescript/lib/typescript.js: Evaluated as low risk
Comments skipped due to low confidence (1)

.github/ISSUE_TEMPLATE/feature.yml:23

  • The word '있다면 여기에 작성해주 세요' should be corrected to '있다면 여기에 작성해 주세요'.
description: 더 전달할 내용이 있다면 여기에 작성해주 세요.
@gwansikk gwansikk changed the title refactor: 프로젝트 구조 변경, 코드 최적화 refactor: 프로젝트 구조 변경 Dec 12, 2024
@gwansikk gwansikk marked this pull request as ready for review December 12, 2024 10:42
@gwansikk gwansikk merged commit 1a1ea8b into main Dec 12, 2024
2 checks passed
@gwansikk gwansikk deleted the refactor/structure branch December 12, 2024 10:42
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.

1 participant