Skip to content

๐Ÿ”ฅ ๋ฆด๋ฆฌ์ฆˆ ์Šน๊ฒฉ ์›Œํฌํ”Œ๋กœ์—์„œ admin ํƒ€๊นƒ ์ œ๊ฑฐ (admin: main ๋ณ‘ํ•ฉ = ํ”„๋กœ๋•์…˜ ๋ฐฐํฌ) - #618

Merged
manNomi merged 2 commits into
mainfrom
chore/remove-prod-release-cd
Aug 1, 2026
Merged

Conversation

@manNomi

@manNomi manNomi commented Aug 1, 2026

Copy link
Copy Markdown
Contributor

์š”์•ฝ

  • Promote Main to Release Branches workflow์—์„œ admin ํƒ€๊นƒ๋งŒ ์ œ๊ฑฐํ•ฉ๋‹ˆ๋‹ค. web/university์˜ release branch ์Šน๊ฒฉ ๋ฐฐํฌ๋Š” ๊ทธ๋Œ€๋กœ ์œ ์ง€๋ฉ๋‹ˆ๋‹ค.
  • admin์€ ์ด์ œ main ๋ณ‘ํ•ฉ๋งŒ์œผ๋กœ ํ”„๋กœ๋•์…˜์ด ๋ฐฐํฌ๋˜๋Š” ๋ชจ๋ธ๋กœ ์ „ํ™˜ํ•ฉ๋‹ˆ๋‹ค. release-admin ์Šน๊ฒฉ ๋‹จ๊ณ„๊ฐ€ ์‚ฌ๋ผ์ง‘๋‹ˆ๋‹ค.
  • docs/git-cicd.md, docs/university-multizone-deployment.md์˜ admin ๋ฐฐํฌ ์„ค๋ช…์„ ๊ฐฑ์‹ ํ–ˆ์Šต๋‹ˆ๋‹ค.

๋ฐฐ๊ฒฝ

์–ด๋“œ๋ฏผ์— API ํ™˜๊ฒฝ(stage/prod) ๋Ÿฐํƒ€์ž„ ์ „ํ™˜์ด ๋„์ž…๋˜๋ฉด์„œ(#615, #616, ํ›„์†์œผ๋กœ ํ”Œ๋กœํŒ… ์Šค์œ„์ฒ˜ ์˜ˆ์ •) ์–ด๋“œ๋ฏผ์„ stage + production ์ด์ค‘์œผ๋กœ ๊ด€๋ฆฌํ•  ํ•„์š”๊ฐ€ ์—†์–ด์กŒ์Šต๋‹ˆ๋‹ค. admin ๋ฐฐํฌ๋ฅผ main ํ•˜๋‚˜๋กœ ํ†ตํ•ฉํ•˜๊ณ , stage/prod API ์ „ํ™˜์€ ์•ฑ ์•ˆ์—์„œ ์ฒ˜๋ฆฌํ•ฉ๋‹ˆ๋‹ค.

ํ•„์š”ํ•œ ํ›„์† ์ž‘์—… (Vercel, ์ €์žฅ์†Œ ์„ค์ •)

  • Vercel Admin Project์˜ Production Branch๋ฅผ release-admin โ†’ main์œผ๋กœ ๋ณ€๊ฒฝ (@hanmw110 ์ง„ํ–‰ ์˜ˆ์ •)
  • ์ „ํ™˜ ํ™•์ธ ํ›„ ์›๊ฒฉ์˜ release-admin ๋ธŒ๋žœ์น˜ ์ •๋ฆฌ(์„ ํƒ)

ํŠน์ด ์‚ฌํ•ญ

  • ์ด PR ๋จธ์ง€ ํ›„ Vercel Admin Project์˜ production branch๋ฅผ ๋ฐ”๊พธ๊ธฐ ์ „๊นŒ์ง€๋Š” release-admin์ด ๋” ์ด์ƒ ๊ฐฑ์‹ ๋˜์ง€ ์•Š์œผ๋ฏ€๋กœ admin ํ”„๋กœ๋•์…˜ ๋ฐฐํฌ ์ˆ˜๋‹จ์ด ์ผ์‹œ์ ์œผ๋กœ ์—†์Šต๋‹ˆ๋‹ค. ๋จธ์ง€ ์งํ›„ Vercel ์„ค์ • ๋ณ€๊ฒฝ์„ ๊ถŒ์žฅํ•ฉ๋‹ˆ๋‹ค.
  • admin์˜ HeadVer ํƒœ๊น…/GitHub Release ์ƒ์„ฑ์€ ์Šน๊ฒฉ workflow ์•ˆ์— ์žˆ์—ˆ์œผ๋ฏ€๋กœ ์ด PR ์ดํ›„ admin ์ž๋™ ํƒœ๊น…์ด ์ค‘๋‹จ๋ฉ๋‹ˆ๋‹ค(web/university ํƒœ๊น…์€ ์œ ์ง€). .github/scripts/create-headver-tag.sh์˜ admin ๋ถ„๊ธฐ๋Š” ๋‹ค๋ฅธ ๊ณณ์—์„œ ์žฌ์‚ฌ์šฉ๋  ์ˆ˜ ์žˆ์–ด ๋‚จ๊ฒจ๋‘์—ˆ์Šต๋‹ˆ๋‹ค.
  • web, university-web์˜ ๋ฐฐํฌ ํ๋ฆ„์€ ๋ณ€๊ฒฝํ•˜์ง€ ์•Š์•˜์Šต๋‹ˆ๋‹ค.

๐Ÿค– Generated with Claude Code

@manNomi
manNomi requested a review from wibaek as a code owner August 1, 2026 16:11
@vercel

vercel Bot commented Aug 1, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
solid-connect-university-web Ready Ready Preview Aug 1, 2026 4:25pm
solid-connect-web-admin Ready Ready Preview Aug 1, 2026 4:25pm
solid-connection-web Ready Ready Preview Aug 1, 2026 4:25pm

@coderabbitai

coderabbitai Bot commented Aug 1, 2026

Copy link
Copy Markdown

Review Change Stack

Walkthrough

  1. ํ”„๋กœ๋•์…˜ ๋ฐฐํฌ ๊ฒฝ๋กœ ๋ณ€๊ฒฝ
    main ๋ธŒ๋žœ์น˜ push๊ฐ€ ๊ฐ Vercel Project์˜ production ๋ฐฐํฌ๋ฅผ ์ง์ ‘ ํŠธ๋ฆฌ๊ฑฐํ•˜๋„๋ก ๋ฌธ์„œ๋ฅผ ๊ฐฑ์‹ ํ–ˆ์Šต๋‹ˆ๋‹ค. ๋ฆด๋ฆฌ์Šค ๋ธŒ๋žœ์น˜ ์Šน๊ฒฉ ์ ˆ์ฐจ์™€ ๊ด€๋ จ workflow ์„ค๋ช…์„ ์ œ๊ฑฐํ–ˆ์Šต๋‹ˆ๋‹ค. .github/workflows/release.yml์„ ์‚ญ์ œํ–ˆ์Šต๋‹ˆ๋‹ค.

Estimated code review effort: 2 (Simple) | ~10 minutes

Suggested reviewers: wibaek, enunsnv, khwww

๐Ÿšฅ Pre-merge checks | โœ… 5
โœ… Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage โœ… Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check โœ… Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check โœ… Passed Check skipped because no linked issues were found for this pull request.
Title check โœ… Passed ์ œ๋ชฉ์€ admin ๋ฆด๋ฆฌ์Šค ๋Œ€์ƒ ์ œ๊ฑฐ๋ฅผ ์„ค๋ช…ํ•˜๋ฉฐ ๋ณ€๊ฒฝ์‚ฌํ•ญ๊ณผ ๊ด€๋ จ๋˜์ง€๋งŒ, ์ „์ฒด ๋ฆด๋ฆฌ์Šค ์›Œํฌํ”Œ๋กœ ์ œ๊ฑฐ๋ฅผ ๋ช…์‹œํ•˜์ง€ ์•Š์Šต๋‹ˆ๋‹ค.
Description check โœ… Passed ์„ค๋ช…์€ ๋ณ€๊ฒฝ ๋‚ด์šฉ, ๋ฐฐ๊ฒฝ, ํ›„์† ์ž‘์—…, ๋ฐฐํฌ ์ค‘๋‹จ ์œ„ํ—˜์„ ํฌํ•จํ•ด ๋Œ€๋ถ€๋ถ„์˜ ํ•„์ˆ˜ ์ •๋ณด๋ฅผ ์ œ๊ณตํ•ฉ๋‹ˆ๋‹ค.
โœจ Finishing Touches
๐Ÿงช Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/remove-prod-release-cd

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.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Actionable comments posted: 3

๐Ÿค– Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@docs/git-cicd.md`:
- Around line 42-44: Update the versioning section in docs/git-cicd.md to
separate headver.jsonโ€™s app-specific head values from HEADVer tag creation
performed by .github/workflows/headver-tagging.yml. Clarify that post-deployment
manual updates and Promote Main to Release Branches/release-* branch promotion
are not part of the current process, rather than documenting them as required
operational steps.

In `@docs/university-multizone-deployment.md`:
- Around line 69-75: The deployment documentation must explicitly require
updating Vercel settings before relying on main-branch production deployments.
In docs/university-multizone-deployment.md lines 69-75, add steps before the
production-branch guidance to set each Vercel Projectโ€™s Settings > Git >
Production Branch to main. In docs/git-cicd.md lines 42-44, state that this
Vercel configuration must be completed before describing main pushes as
triggering production deployments.
- Around line 69-75: ๋ฌธ์„œ์˜ Vercel production ๋ฐฐํฌ ์„ค๋ช…์— ๊ฐ ํ”„๋กœ์ ํŠธ์˜ ์„ค์ • ์ ˆ์ฐจ๋ฅผ ์ถ”๊ฐ€ํ•˜์„ธ์š”. Main
Web, Admin, University Web Project์—์„œ ๊ฐ๊ฐ Settings > Git์˜ Production Branch๋ฅผ
main์œผ๋กœ ์ง€์ •ํ•˜๊ณ , ์„ค์ • ์ „ํ™˜ ํ›„ main push๊ฐ€ production ๋ฐฐํฌ๋ฅผ ํŠธ๋ฆฌ๊ฑฐํ•˜๋Š”์ง€ ํ™•์ธํ•˜๋Š” ๋‹จ๊ณ„๊นŒ์ง€ ๋ช…์‹œํ•˜์„ธ์š”.
๐Ÿช„ Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

โ„น๏ธ Review info
โš™๏ธ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: b52f431e-83cc-49eb-ac75-bd764c1b61d0

๐Ÿ“ฅ Commits

Reviewing files that changed from the base of the PR and between 97f1cb0 and 65ffeff.

๐Ÿ“’ Files selected for processing (3)
  • .github/workflows/release.yml
  • docs/git-cicd.md
  • docs/university-multizone-deployment.md
๐Ÿ’ค Files with no reviewable changes (1)
  • .github/workflows/release.yml

Comment thread docs/git-cicd.md Outdated
Comment on lines +42 to +44
main ๋ธŒ๋žœ์น˜์— push๊ฐ€ ๋˜๋ฉด ์ž๋™์œผ๋กœ CI๊ฐ€ ์ง„ํ–‰๋˜๊ณ , ๊ฐ Vercel Project์˜ production ํ™˜๊ฒฝ์— ๋ฐฐํฌ๋ฉ๋‹ˆ๋‹ค. ๊ฐ Vercel Project์˜ production branch๋Š” `main`์ž…๋‹ˆ๋‹ค.

Github Actions์˜ `Promote Main to Release Branches`๋ฅผ ๊ฐ€๋™์‹œํ‚ค๋ฉด ์„ ํƒํ•œ release branch๊ฐ€ main์œผ๋กœ ๊ฐฑ์‹ ๋˜๊ณ , ๊ฐ Vercel Project์˜ production ํ™˜๊ฒฝ์— ๋ฐฐํฌ๋ฉ๋‹ˆ๋‹ค.

- `release-web`: web production ๋ฐฐํฌ
- `release-admin`: admin production ๋ฐฐํฌ
- `release-university`: university-web production ๋ฐฐํฌ

๋ฆด๋ฆฌ์ฆˆ workflow์˜ target์€ ๋‹ค์Œ๊ณผ ๊ฐ™์ด ์‚ฌ์šฉํ•ฉ๋‹ˆ๋‹ค.

- `all`: web, admin, university-web ์ „์ฒด ๋ฆด๋ฆฌ์ฆˆ
- `web`: web๋งŒ ๋ฆด๋ฆฌ์ฆˆ
- `admin`: admin๋งŒ ๋ฆด๋ฆฌ์ฆˆ
- `university`: university-web๋งŒ ๋ฆด๋ฆฌ์ฆˆ

๊ธฐ๋ณธ๊ฐ’์€ `all`์ž…๋‹ˆ๋‹ค. ํŠน์ • project๋งŒ production ๋ฆด๋ฆฌ์ฆˆํ•˜๋ ค๋ฉด `web`, `admin`, `university`๋ฅผ ์„ ํƒํ•ฉ๋‹ˆ๋‹ค.
๊ณผ๊ฑฐ์—๋Š” main ๋ณ‘ํ•ฉ ํ›„ GitHub Actions์˜ `Promote Main to Release Branches`๋กœ release branch(`release-web`/`release-admin`/`release-university`)๋ฅผ ๊ฐฑ์‹ ํ•ด์•ผ production์ด ๋ฐฐํฌ๋˜์—ˆ์œผ๋‚˜, ํ•ด๋‹น workflow์™€ release branch ์Šน๊ฒฉ ๋‹จ๊ณ„๋Š” ์ œ๊ฑฐ๋˜์—ˆ์Šต๋‹ˆ๋‹ค. main ๋ณ‘ํ•ฉ์ด ๊ณง production ๋ฐฐํฌ์ž…๋‹ˆ๋‹ค.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

๐Ÿ—„๏ธ Data Integrity & Integration | ๐ŸŸ  Major | ๐Ÿ—๏ธ Heavy lift

๐Ÿงฉ Analysis chain

๐Ÿ Script executed:

#!/bin/bash
set -euo pipefail

echo "== files =="
git ls-files | sed -n '1,120p'

echo "== docs/git-cicd.md lines 1-90 =="
cat -n docs/git-cicd.md | sed -n '1,90p'

echo "== workflow/headver/release references =="
rg -n "HeadVer|headver|release branch|release-web|release-admin|release-university|GitHub Release|Promote Main|create.*release|Create Release|gh release|release" -S . --glob '!node_modules' --glob '!dist' --glob '!build' | sed -n '1,240p'

Repository: solid-connection/solid-connect-web

Length of output: 8249


๐Ÿ Script executed:

#!/bin/bash
set -euo pipefail

echo "== workflow files =="
cat -n .github/workflows/ci.yml
echo "---"
cat -n .github/workflows/headver-tagging.yml

echo "== scripts =="
cat -n .github/scripts/create-headver-tag.sh

echo "== headver.json tracked/config =="
for f in $(git ls-files | grep -E '(^|/)headver\.json$|^pnpm-workspace\.yaml$|^package\.json$'); do
  echo "--- $f"
  sed -n '1,120p' "$f"
done

echo "== docs onboarding lines 45-58 =="
cat -n docs/onboarding.md | sed -n '45,58p'

Repository: solid-connection/solid-connect-web

Length of output: 14206


1. HeadVer ํƒœ๊น…๊ณผ release branch ๊ณผ์ •์„ ๋ฌธ์„œ์—์„œ ๋ถ„๋ฆฌํ•˜์„ธ์š”.

docs/git-cicd.md์˜ ๋ฒ„์ €๋‹ ์„น์…˜์ด ํ˜„์žฌ CI ์ƒํƒœ์™€ ๋งž์ง€ ์•Š์Šต๋‹ˆ๋‹ค. headver.json์€ ํ˜„์žฌ ์•ฑ๋ณ„ head ๊ฐ’๋งŒ ์ €์žฅ๋˜๊ณ , HEADVer ํƒœ๊น…์€ .github/workflows/headver-tagging.yml์—์„œ ๋ณ„๋„ ํ˜ธ์ถœํ•ด ์ƒ์„ฑํ•ฉ๋‹ˆ๋‹ค. ๋ฐฐํฌ ์ดํ›„ ์ˆ˜๋™ ์—…๋ฐ์ดํŠธ๋‚˜ Promote Main to Release Branches/release-* branch ๊ฐฑ์‹  ๊ณผ์ •์„ ์šด์˜ ์ ˆ์ฐจ์— ์ถ”๊ฐ€ํ•˜๊ฑฐ๋‚˜, ํ˜„์žฌ ์ ˆ์ฐจ๊ฐ€ ์•„๋‹˜์„ ๋ถ„๋ช…ํžˆ ์ ์–ด์ฃผ์„ธ์š”.

๐Ÿค– Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@docs/git-cicd.md` around lines 42 - 44, Update the versioning section in
docs/git-cicd.md to separate headver.jsonโ€™s app-specific head values from
HEADVer tag creation performed by .github/workflows/headver-tagging.yml. Clarify
that post-deployment manual updates and Promote Main to Release
Branches/release-* branch promotion are not part of the current process, rather
than documenting them as required operational steps.

Comment thread docs/university-multizone-deployment.md Outdated
Comment on lines +69 to +75
production ๋ฐฐํฌ๋Š” main ๋ธŒ๋žœ์น˜ ๋ณ‘ํ•ฉ์œผ๋กœ ํŠธ๋ฆฌ๊ฑฐ๋œ๋‹ค. ๊ฐ Vercel Project์˜ production branch๋Š” `main`์ด๋ฉฐ, main์— push๋˜๋ฉด ํ•ด๋‹น project์˜ production ํ™˜๊ฒฝ์— ๋ฐฐํฌ๋œ๋‹ค.

- Main Web Project production branch: `release-web`
- Admin Project production branch: `release-admin`
- University Web Project production branch: `release-university`
- Main Web Project production branch: `main`
- Admin Project production branch: `main`
- University Web Project production branch: `main`

workflow target์€ ๋‹ค์Œ๊ณผ ๊ฐ™์ด ์‚ฌ์šฉํ•œ๋‹ค.

- `all`: `release-web`, `release-admin`, `release-university`๋ฅผ ๋ชจ๋‘ main์œผ๋กœ ๊ฐฑ์‹ 
- `web`: `release-web`๋งŒ main์œผ๋กœ ๊ฐฑ์‹ 
- `admin`: `release-admin`๋งŒ main์œผ๋กœ ๊ฐฑ์‹ 
- `university`: `release-university`๋งŒ main์œผ๋กœ ๊ฐฑ์‹ 

workflow ๊ธฐ๋ณธ๊ฐ’์€ `all`์ด๋‹ค. ๊ฐœ๋ณ„ production ๋ฐฐํฌ๊ฐ€ ํ•„์š”ํ•˜๋ฉด `web`, `admin`, `university`๋ฅผ ์„ ํƒํ•œ๋‹ค.
(๊ณผ๊ฑฐ์—๋Š” `Promote Main to Release Branches` workflow๋กœ `release-web`/`release-admin`/`release-university` ๋ธŒ๋žœ์น˜๋ฅผ ๊ฐฑ์‹ ํ•ด production ๋ฐฐํฌ๋ฅผ ํŠธ๋ฆฌ๊ฑฐํ–ˆ์œผ๋‚˜, ํ•ด๋‹น workflow์™€ release branch ์Šน๊ฒฉ ๋‹จ๊ณ„๋Š” ์ œ๊ฑฐ๋˜์—ˆ๋‹ค.)

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

๐Ÿฉบ Stability & Availability | ๐ŸŸ  Major | ๐Ÿ—๏ธ Heavy lift

๐Ÿงฉ Analysis chain

๐Ÿ Script executed:

#!/bin/bash
set -euo pipefail

echo "== files =="
git ls-files | rg '(^docs/.*multizone|^docs/git-cicd\.md$|^vercel\.json$|github\.workflows/.+\.(ya?ml))' || true

echo
echo "== docs snippets =="
sed -n '55,85p' docs/university-multizone-deployment.md 2>/dev/null || true
echo "---"
sed -n '32,52p' docs/git-cicd.md 2>/dev/null || true

echo
echo "== workflow references =="
rg -n "Promote Main to Release Branches|release-(web|admin|university)|production branch|productionBranch|Deployment|vercel|project" .github docs vercel.json 2>/dev/null || true

echo
echo "== vercel config =="
cat -n vercel.json 2>/dev/null || true

Repository: solid-connection/solid-connect-web

Length of output: 7254


1. Vercel Production Branch ์ „ํ™˜์„ ๋ฐฐํฌ ์ „์ œ์กฐ๊ฑด์œผ๋กœ ๋ช…์‹œํ•˜์„ธ์š”.

Vercel productionBranch๋Š” ์ €์žฅ์†Œ ํŒŒ์ผ์ด ์•„๋‹ˆ๋ผ ๊ฐ Vercel Project ์„ค์ •์œผ๋กœ ์ •ํ•ด์ง‘๋‹ˆ๋‹ค.

1) university-multizone-deployment.md์˜ production branch ์•ˆ๋‚ด ์•ž๋‹จ์—์„œ ๊ฐ Project์˜ Vercel Settings > Git > Production Branch๋ฅผ `main`์œผ๋กœ ๋ณ€๊ฒฝํ•˜๋Š” ์ˆœ์„œ๋ฅผ ์ถ”๊ฐ€ํ•˜์„ธ์š”.
2) git-cicd.md์˜ `main` push โ†’ production ๋ฐฐํฌ ์„ค๋ช… ์•ž์— Vercel ์„ค์ • ์™„๋ฃŒ ์กฐ๊ฑด์„ ๋ช…์‹œํ•˜์„ธ์š”.
๐Ÿ“ Affects 2 files
  • docs/university-multizone-deployment.md#L69-L75 (this comment)
  • docs/git-cicd.md#L42-L44
๐Ÿค– Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@docs/university-multizone-deployment.md` around lines 69 - 75, The deployment
documentation must explicitly require updating Vercel settings before relying on
main-branch production deployments. In docs/university-multizone-deployment.md
lines 69-75, add steps before the production-branch guidance to set each Vercel
Projectโ€™s Settings > Git > Production Branch to main. In docs/git-cicd.md lines
42-44, state that this Vercel configuration must be completed before describing
main pushes as triggering production deployments.

๐Ÿฉบ Stability & Availability | ๐ŸŸ  Major | ๐Ÿ—๏ธ Heavy lift

๐Ÿงฉ Analysis chain

๐Ÿ Script executed:

#!/bin/bash
set -euo pipefail

echo "== files =="
git ls-files | sed -n '1,120p'

echo
echo "== target doc excerpt =="
if [ -f docs/university-multizome-deployment.md ]; then
  sed -n '1,120p' docs/university-multizone-deployment.md
else
  fd -a 'university-multizone-deployment\.md' . | while read -r f; do
    echo "-- $f"
    sed -n '1,120p' "$f"
  done
fi

echo
echo "== search vercel deployment docs/workflows =="
rg -n "Vercel|production branch|Production Branch|Vercl|release-web|release-admin|release-university|Promote Main to Release Branches|็”Ÿไบง|๋ฐฐํฌ" -S . --glob '!node_modules' --glob '!dist' --glob '!build' | sed -n '1,220p'

Repository: solid-connection/solid-connect-web

Length of output: 13736


๐Ÿ Script executed:

#!/bin/bash
set -euo pipefail

echo "== workflows =="
for f in .github/workflows/*.yml .github/workflows/*.yaml; do
  [ -f "$f" ] || continue
  echo "-- $f"
  wc -l "$f"
  sed -n '1,220p' "$f" | sed -n '1,220p'
done

echo
echo "== git-cicd relevant =="
sed -n '1,90p' docs/git-cicd.md

echo
echo "== search release workflow files in history briefly =="
git ls-files .github/workflows | xargs -r -I{} sh -c 'echo "== {} =="; sed -n "1,220p" "{}" | grep -nE "release|release-(web|admin|university)|Promote|deploy|push_to_release|production branch|production branch" || true'

Repository: solid-connection/solid-connect-web

Length of output: 21343


1. Vercel Production Branch ์„ค์ • ์ ˆ์ฐจ๋ฅผ ๋ฌธ์„œ์— ์ถ”๊ฐ€ํ•˜์„ธ์š”.

๋ฌธ์„œ๋Š” production branch๊ฐ€ ์ด๋ฏธ main์ด๋ผ๊ณ  ์„ค๋ช…ํ•˜์ง€๋งŒ, ๋ฐฐํฌ ์›Œํฌํ”Œ๋กœ๋Š” release ๋ธŒ๋žœ์น˜ ๊ฐฑ์‹ ์„ ์ œ๊ฑฐํ–ˆ์„ ๋ฟ Vercel ์„ค์ •์„ ๋Œ€์ฒดํ•˜์ง€ ์•Š์Šต๋‹ˆ๋‹ค.

- ๊ฐ ํ”„๋กœ์ ํŠธ์—์„œ `Settings > Git`์˜ Production Branch๋ฅผ `main`์œผ๋กœ ์„ค์ •ํ•ฉ๋‹ˆ๋‹ค.
- ์ „ํ™˜ ํ›„ `main` push๋กœ ์ƒ์‚ฐ ํ™˜๊ฒฝ์— ๋ฐฐํฌ๋˜๋Š”์ง€ ๊ฐ ํ”„๋กœ์ ํŠธ์—์„œ ํ™•์ธํ•ฉ๋‹ˆ๋‹ค.
๐Ÿค– Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@docs/university-multizone-deployment.md` around lines 69 - 75, ๋ฌธ์„œ์˜ Vercel
production ๋ฐฐํฌ ์„ค๋ช…์— ๊ฐ ํ”„๋กœ์ ํŠธ์˜ ์„ค์ • ์ ˆ์ฐจ๋ฅผ ์ถ”๊ฐ€ํ•˜์„ธ์š”. Main Web, Admin, University Web
Project์—์„œ ๊ฐ๊ฐ Settings > Git์˜ Production Branch๋ฅผ main์œผ๋กœ ์ง€์ •ํ•˜๊ณ , ์„ค์ • ์ „ํ™˜ ํ›„ main push๊ฐ€
production ๋ฐฐํฌ๋ฅผ ํŠธ๋ฆฌ๊ฑฐํ•˜๋Š”์ง€ ํ™•์ธํ•˜๋Š” ๋‹จ๊ณ„๊นŒ์ง€ ๋ช…์‹œํ•˜์„ธ์š”.

@manNomi
manNomi force-pushed the chore/remove-prod-release-cd branch from 65ffeff to 474a5ae Compare August 1, 2026 16:21
@manNomi manNomi changed the title ๐Ÿ”ฅ ํ”„๋กœ๋•์…˜ ๋ฆด๋ฆฌ์ฆˆ ์Šน๊ฒฉ ์›Œํฌํ”Œ๋กœ ์ œ๊ฑฐ (main ๋ณ‘ํ•ฉ = ํ”„๋กœ๋•์…˜ ๋ฐฐํฌ) ๐Ÿ”ฅ ๋ฆด๋ฆฌ์ฆˆ ์Šน๊ฒฉ ์›Œํฌํ”Œ๋กœ์—์„œ admin ํƒ€๊นƒ ์ œ๊ฑฐ (admin: main ๋ณ‘ํ•ฉ = ํ”„๋กœ๋•์…˜ ๋ฐฐํฌ) Aug 1, 2026
@manNomi
manNomi merged commit 8f2c130 into main Aug 1, 2026
13 checks passed
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