Conversation
Summary of ChangesHello @chunjaemin, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! 이 PR은 프로젝트의 초기 환경을 견고하게 설정하고, 확장 가능한 아키텍처를 도입하는 데 중점을 둡니다. 주요 변경 사항으로는 Highlights
Changelog
Ignored Files
Activity
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
| * 날짜를 `YYYY.MM.DD` 형태로 포맷합니다. (기본: ko-KR) | ||
| * | ||
| * - Date, timestamp(number), ISO string(string) 모두 허용 | ||
| * - invalid date면 빈 문자열 반환 | ||
| */ | ||
| export function formatDate( | ||
| input: DateInput, | ||
| options?: { | ||
| locale?: string; | ||
| separator?: "." | "-" | "/"; | ||
| } | ||
| ): string { |
There was a problem hiding this comment.
formatDate 함수의 주석에는 (기본: ko-KR)이라고 명시되어 있고, options으로 locale을 받지만 실제 구현에서는 사용되지 않고 있습니다. 현재 구현은 getFullYear, getMonth, getDate를 사용하여 문자열을 직접 조합하므로 로케일에 영향을 받지 않습니다. 이는 버그의 소지가 있거나 최소한 오해를 불러일으킬 수 있습니다.
locale 옵션을 제거하고 주석을 수정하여 현재 구현과 일치시키는 것을 제안합니다.
* 날짜를 `YYYY.MM.DD` 형태로 포맷합니다.
*
* - Date, timestamp(number), ISO string(string) 모두 허용
* - invalid date면 빈 문자열 반환
*/
export function formatDate(
input: DateInput,
options?: {
separator?: "." | "-" | "/";
}
): string {| "eslint": "^9.25.0", | ||
| "eslint-config-expo": "~10.0.0", | ||
| "prettier-plugin-tailwindcss": "^0.7.2", |
There was a problem hiding this comment.
biome을 린터 및 포맷터로 사용하고 있으므로, devDependencies에 있는 eslint, eslint-config-expo, prettier-plugin-tailwindcss 패키지들은 현재 설정에서 사용되지 않는 것으로 보입니다. package.json의 스크립트에서도 biome만 사용되고 있습니다. 사용하지 않는 의존성을 제거하여 프로젝트를 더 깔끔하게 유지하는 것이 좋겠습니다.
| @@ -0,0 +1,182 @@ | |||
| import { ScrollView, Text, View } from "react-native"; | |||
| import { shadows } from "../../shared/styles/shadows"; | |||
There was a problem hiding this comment.
tsconfig.json에 경로 별칭(path alias)이 @/*로 설정되어 있습니다. 프로젝트의 일관성을 위해 상대 경로 대신 별칭을 사용하는 것이 좋습니다.
| import { shadows } from "../../shared/styles/shadows"; | |
| import { shadows } from "@/shared/styles/shadows"; |
| .dark { | ||
| /* Primitive Colors - 원시값 (지정된 색상만) - 현재는 root와 동일 */ | ||
|
|
||
| /* 파란색 계열 */ | ||
| --blue-100: #e5f6fe; | ||
| --blue-50: #f4f6fe; | ||
| --blue-500: #0068fe; | ||
|
|
||
| /* 회색 계열 */ | ||
| --gray-100: #f4f4f5; | ||
| --gray-300: #dbdde1; | ||
| --gray-500: #8e9398; | ||
|
|
||
| /* 흰색 */ | ||
| --white: #fefffe; | ||
|
|
||
| /* 검은색 */ | ||
| --black: #040404; | ||
|
|
||
| /* 빨간색 */ | ||
| --red-500: #ff6562; | ||
|
|
||
| /* Shadow Colors - Primitive */ | ||
| --blue-shadow-rgb: 102 164 254; | ||
| --gray-shadow-rgb: 142 147 152; | ||
|
|
||
| /* Brand Colors - Primitive 참조 */ | ||
| --color-primary: var(--blue-500); | ||
| --color-primary-tint: var(--blue-100); | ||
|
|
||
| /* Neutral Colors - Primitive 참조 */ | ||
| --color-neutral: var(--gray-100); | ||
| --color-neutral-variant: var(--gray-300); | ||
|
|
||
| /* Background Colors - Primitive 참조 */ | ||
| --color-canvas: var(--white); | ||
|
|
||
| /* State Colors - Primitive 참조 */ | ||
| --color-danger: var(--red-500); | ||
|
|
||
| /* Text Colors - Primitive 참조 */ | ||
| --color-content-primary: var(--black); | ||
| --color-content-secondary: var(--gray-500); | ||
| --color-content-inverse: var(--blue-50); | ||
|
|
||
| /* Layout Spacing */ | ||
| --layout-margin: 24px; /* 화면 좌우 전체 여백 */ | ||
| --layout-gutter: 10px; /* 그리드/리스트 요소 사이의 간격 */ | ||
|
|
||
| /* Shadow (X Y Blur Spread Color) */ | ||
| --shadow-primary: 0px 0px 6px 0px rgba(var(--blue-shadow-rgb), 0.4); /* blue-shadow의 40% 투명도 */ | ||
| --shadow-neutral: 0px 5px 10px 0px rgba(var(--gray-shadow-rgb), 0.15); /* gray-shadow의 15% 투명도 */ | ||
| } |
There was a problem hiding this comment.
.dark 클래스 내의 CSS 변수들이 :root에 정의된 변수들과 완전히 동일하게 중복 선언되어 있습니다. 다크 모드에서 값이 변경되는 변수들만 .dark 내에 선언하고, 나머지는 제거하여 중복을 줄이는 것이 유지보수 관점에서 더 좋을 것 같습니다. 현재는 변경되는 값이 없으므로, 향후 다크 모드 구현을 위한 주석만 남겨두는 것을 제안합니다.
.dark {
/* TODO: 다크 모드에서 재정의할 색상 변수를 여기에 추가합니다. */
/* 예시: */
/* --color-canvas: var(--black); */
/* --color-content-primary: var(--white); */
}
|
확인했습니다 수고하셨어요👏👏 |
3 participants
추가 작업 내용
이전 프로젝트 파일(#3브랜치 2/11일 시점)에 존재했던 몇가지 개선점을 보완하였습니다.
주요 변경 사항
README.md 수정
폰트 import 경로 수정
프리텐다드 폰트 굵기 (커스텀 클래스) 추가
사용 예시
font-regularfont-mediumfont-semiboldfont-bold