Skip to content

ci(global): PR마다 web·admin 타입체크와 빌드를 검증하는 워크플로 추가 - #48

Open
LSH-0125 wants to merge 1 commit into
mainfrom
ci/global/typecheck-build-workflow
Open

LSH-0125 wants to merge 1 commit into
mainfrom
ci/global/typecheck-build-workflow

Conversation

@LSH-0125

@LSH-0125 LSH-0125 commented Sep 16, 2026

Copy link
Copy Markdown

web 저장소에는 CI가 없어 PR 머지 전 자동 검증이 없었음.
Vercel 빌드도 next.config.mjs의 ignoreBuildErrors와 admin의 tsc --noCheck 때문에 타입 에러를 걸러 주지 않음. (근데 검증 결과 타입 에러는 없긴 했음)
main 머지가 곧 프로덕션 배포라 검증 없이 배포되는 구조였다.

PR과 main 푸시마다 두 잡을 돌린다.

  • web: next buildtsc --noEmit. tsconfig의 include에 빌드가 만드는 next-env.d.ts와 .next/types가 들어 있어 빌드를 먼저 돌려야 한다.
  • admin: tsc -bvite build. build 스크립트가 tsc -b --noCheck라 타입을 검사하지 않으므로 여기서 --noCheck 없이 따로 돌린다.

두 잡의 이름은 브랜치 룰셋에서 required status check으로 지정할 이름과
같게 맞췄다.

함께 .nvmrc를 18에서 22로 올린다. Next 15가 요구하는 최소 버전이 18.18이라 기존 값은 하한을 만족하지 못하고, Node 18은 지원이 끝났다. 워크플로가 node-version-file로 이 파일을 읽는다.

변경 사항

관련 이슈

  • closes #

테스트

  • 로컬 빌드/실행 확인
  • 영향 범위 검토 완료

참고

web 저장소에는 CI가 없어 PR 머지 전 자동 검증이 전혀 없었다. Vercel 빌드도
next.config.mjs의 ignoreBuildErrors와 admin의 `tsc --noCheck` 때문에 타입
에러를 걸러 주지 않는다. main 머지가 곧 프로덕션 배포라 검증 없이 배포되는
구조였다.

PR과 main 푸시마다 두 잡을 돌린다.

- web: `next build` 후 `tsc --noEmit`. tsconfig의 include에 빌드가 만드는
  next-env.d.ts와 .next/types가 들어 있어 빌드를 먼저 돌려야 한다.
- admin: `tsc -b` 후 `vite build`. build 스크립트가 `tsc -b --noCheck`라
  타입을 검사하지 않으므로 여기서 --noCheck 없이 따로 돌린다.

두 잡의 이름은 브랜치 룰셋에서 required status check으로 지정할 이름과
같게 맞췄다.

함께 .nvmrc를 18에서 22로 올린다. Next 15가 요구하는 최소 버전이 18.18이라
기존 값은 하한을 만족하지 못하고, Node 18은 지원이 끝났다. 워크플로가
node-version-file로 이 파일을 읽는다.
@vercel

vercel Bot commented Sep 16, 2026

Copy link
Copy Markdown

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

2 Skipped Deployments
Project Deployment Actions Updated
khuda-homepage-web Ignored Ignored Sep 16, 2026 10:55am UTC
khuda-homepage-web-admin Ignored Ignored Sep 16, 2026 10:55am UTC

This branch has not been deployed

No deployments
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