Skip to content

fix: deploy error#212

Merged
iOdiO89 merged 1 commit into
devfrom
fix/deploy
Jun 16, 2026
Merged

fix: deploy error#212
iOdiO89 merged 1 commit into
devfrom
fix/deploy

Conversation

@iOdiO89

@iOdiO89 iOdiO89 commented Jun 16, 2026

Copy link
Copy Markdown
Contributor

작업 요약

해당 PR에서 작업한 내용을 정리해주세요.
Slack 요약에는 1단계 불릿만 반영됩니다. 예) - 로그인 API를 연동합니다

작업 세부 내용

스크린샷

연관 이슈

이 PR과 연관된 이슈 번호를 작성하세요. 예) closes #10

Summary by CodeRabbit

릴리스 노트

  • Chores
    • 이미지 원격 패턴 설정을 간소화했습니다.
    • 패키지 구성을 업데이트했습니다.

@vercel

vercel Bot commented Jun 16, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
depromeet Ready Ready Preview, Comment Jun 16, 2026 12:05pm

@coderabbitai

coderabbitai Bot commented Jun 16, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: f7a601ed-deb1-44cd-8561-b5e21a79e949

📥 Commits

Reviewing files that changed from the base of the PR and between 70083b7 and 4464843.

📒 Files selected for processing (2)
  • apps/web/next.config.mjs
  • apps/web/package.json
💤 Files with no reviewable changes (2)
  • apps/web/next.config.mjs
  • apps/web/package.json

Walkthrough

apps/web/next.config.mjs에서 이미지 원격 패턴 중 HTTP, picsum.photos 등 임시 항목을 제거하고 HTTPS 와일드카드 단일 패턴만 유지합니다. apps/web/package.json에서는 "type": "module" 필드가 삭제됩니다.

Changes

Web 앱 설정 정리

Layer / File(s) Summary
이미지 패턴 및 모듈 타입 설정 제거
apps/web/next.config.mjs, apps/web/package.json
images.remotePatterns에서 HTTP 및 picsum.photos 등 임시 패턴을 제거하고 protocol: 'https', hostname: '**' 단일 패턴만 유지합니다. package.json에서 "type": "module" 선언을 삭제합니다.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐇 토끼가 코드 밭을 깔끔히 정돈했어요
쓸데없는 패턴들은 모두 뽑아냈죠
HTTP와 picsum은 이제 안녕~
"type": "module" 도 조용히 사라지고
HTTPS 하나로 깔끔하게! 🌿

🚥 Pre-merge checks | ✅ 2 | ❌ 3

❌ Failed checks (2 warnings, 1 inconclusive)

Check name Status Explanation Resolution
Linked Issues check ⚠️ Warning 이 PR의 변경사항(next.config.mjs, package.json 수정)은 연결된 이슈 #10(모바일 앱 기초 설정)의 목표와 불일치합니다. 이슈 #10은 React Native/Expo 앱 프로젝트 구성이 목표인데, 이 PR은 web 앱 설정을 수정합니다. PR이 실제로 해결하는 이슈를 명확히 확인하고, 올바른 이슈를 링크하거나 이슈를 새로 생성하세요.
Out of Scope Changes check ⚠️ Warning 이 PR의 모든 변경사항(next.config.mjs의 이미지 패턴 제거, package.json의 module 필드 제거)이 연결된 이슈 #10의 범위 외입니다. 이슈 #10의 목표는 모바일 앱 설정이지만, PR은 웹 앱 설정을 수정합니다. 이 변경사항들과 관련된 새로운 이슈를 생성하고 해당 이슈를 링크하거나, PR 설명에서 변경사항이 필요한 이유를 명확히 설명하세요.
Title check ❓ Inconclusive PR 제목 'fix: deploy error'는 배포 오류를 수정한다는 의도를 나타내지만, 실제 변경사항(next.config.mjs의 이미지 패턴 제거, package.json의 module 필드 제거)과의 직접적인 관련성이 명확하지 않습니다. PR 제목을 변경사항을 더 구체적으로 설명하는 것으로 수정하세요. 예: 'fix: remove dev image patterns and module type from web app config'
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/deploy

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions
github-actions Bot requested review from kanghaeun and ychany June 16, 2026 12:05
@github-actions github-actions Bot added the fix Something isn't working label Jun 16, 2026
@github-actions

Copy link
Copy Markdown

No description provided.

@iOdiO89
iOdiO89 merged commit 9f63c06 into dev Jun 16, 2026
8 checks passed
@iOdiO89
iOdiO89 deleted the fix/deploy branch June 16, 2026 12:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fix Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant