Skip to content

fix: release 배포에서 vercel link 스코프 의존성 제거#434

Merged
manNomi merged 1 commit intomainfrom
fix/release-vercel-scope-and-react19-ref
Feb 16, 2026
Merged

fix: release 배포에서 vercel link 스코프 의존성 제거#434
manNomi merged 1 commit intomainfrom
fix/release-vercel-scope-and-react19-ref

Conversation

@manNomi
Copy link
Contributor

@manNomi manNomi commented Feb 16, 2026

Summary

  • release workflow에서 vercel link --scope=... 단계를 제거해 scope 불일치로 인한 배포 실패를 방지했습니다.
  • vercel pull/vercel build/vercel deploy 흐름은 유지하여 기존 배포 동작은 그대로 보존했습니다.
  • ci:check를 로컬에서 실행해 타입체크/린트 통과를 확인했습니다.

@vercel
Copy link

vercel bot commented Feb 16, 2026

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

Project Deployment Actions Updated (UTC)
solid-connect-web-admin Ready Ready Preview, Comment Feb 16, 2026 9:15am
solid-connection-web Ready Ready Preview, Comment Feb 16, 2026 9:15am

@coderabbitai
Copy link

coderabbitai bot commented Feb 16, 2026

No actionable comments were generated in the recent review. 🎉


Walkthrough

  1. Vercel 프로젝트 링크 단계 제거
    • deploy_production 작업에서 Vercel 프로젝트 연결 단계(vercel link)가 삭제되었습니다. 이 변경으로 환경 정보 조회 및 빌드/배포 작업 이전의 명시적 프로젝트 연결 프로세스가 더 이상 실행되지 않습니다.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Suggested reviewers

  • wibaek
  • enunsnv
  • khwww
🚥 Pre-merge checks | ✅ 3 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Merge Conflict Detection ⚠️ Warning ❌ Merge conflicts detected (3 files):

⚔️ .github/workflows/release.yml (content)
⚔️ apps/admin/package.json (content)
⚔️ package.json (content)

These conflicts must be resolved before merging into main.
Resolve conflicts locally and push changes to this branch.
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed PR 제목이 변경사항의 핵심을 명확하게 반영하고 있습니다: Vercel link 스코프 의존성 제거를 통한 배포 수정.
Description check ✅ Passed PR 설명이 작업 내용을 충분히 설명하고 있습니다: 제거된 단계, 보존된 흐름, 로컬 검증이 명시되어 있습니다.
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 unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/release-vercel-scope-and-react19-ref
⚔️ Resolve merge conflicts (beta)
  • Auto-commit resolved conflicts to branch fix/release-vercel-scope-and-react19-ref
  • Create stacked PR with resolved conflicts
  • Post resolved changes as copyable diffs in a comment

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.

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