-
Notifications
You must be signed in to change notification settings - Fork 1
[25.01.01 / TASK-80] Feature - 배포 준비 #7
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
Conversation
워크스루이 풀 리퀘스트는 환경 설정 파일과 API 요청 인스턴스의 오류 처리를 개선합니다. 변경 사항
토끼의 시
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
🧹 Nitpick comments (2)
src/apis/instance.request.ts (2)
70-70
: 에러 파라미터를unknown
으로 처리하는 방식은 타입 안정성을 높여줍니다.
unknown
타입 사용을 통해 런타임 시점에서 보다 안전하게 에러를 처리할 수 있게 되었습니다. 다만,location
객체는 브라우저 환경에서만 존재하므로, SSR 환경에서 예외가 발생하지 않도록 주의하는 것이 좋습니다.
90-90
: 에러 메시지에 에러 객체의 이름을 삽입할 때는 에러 타입이 올바른지 다시 확인해주세요.
(err as Error).name
을 사용하는 것은 유효하지만, 실제 에러가Error
타입이 아닐 수도 있습니다. 에러 타입별 처리를 고려하거나,err instanceof Error
체크를 통해 안정성을 높이는 방법을 검토해 보세요.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (3)
.env.sample
(1 hunks).gitignore
(1 hunks)src/apis/instance.request.ts
(2 hunks)
✅ Files skipped from review due to trivial changes (1)
- .env.sample
🔇 Additional comments (1)
.gitignore (1)
38-39
: 환경 파일을 명확히 구분하는 것은 배포 환경 관리에 유용합니다.
개발용, 운영용 환경 파일을 별도로 관리함으로써 환경 설정 충돌을 줄이고, 스테이징/프로덕션 등 추가 환경의 변경에도 확장성이 좋아집니다.
Summary by CodeRabbit
구성
.env.sample
파일에NEXT_PUBLIC_BASE_URL
,NEXT_PUBLIC_ABORT_MS
,SENTRY_AUTH_TOKEN
,SENTRY_DSN
환경 변수 포함개선
.gitignore
파일 업데이트:.env
대신.env.development
와.env.production
추가기술적 개선