Skip to content

Conversation

@Seungwoo321
Copy link
Member

🚨 3번째 시도 - 베타 릴리스 실패 근본 원인 해결

문제점

  1. 빈 태그 'v' 생성: step ID 누락으로 ${{ steps.version.outputs.version }} 빈 값 반환
  2. 베타 중복: 1.1.1-beta.123-beta.456 형태로 베타가 계속 추가됨
  3. GitHub Release 실패 시에도 npm 퍼블리시 진행

해결방법

1️⃣ step ID 추가

- name: Version packages as beta
  id: version  # ✅ 추가됨

2️⃣ 베타 중복 방지 로직 개선

# Before: 마지막 베타만 제거
sed 's/-beta\.[0-9]*$//'

# After: 모든 베타 제거
sed 's/-beta\.[0-9]*//g'

3️⃣ 릴리스 순서 변경

# Before: npm 퍼블리시 → GitHub Release
# After: GitHub Release → npm 퍼블리시 (성공시에만)

- name: Create GitHub Pre-release
  id: create-release
  
- name: Publish pre-release to npm  
  if: steps.create-release.outcome == 'success'

4️⃣ 태그명 표준화

tag_name: vue-pivottable@${{ steps.version.outputs.version }}

테스트 결과

# 베타 중복 제거 테스트
echo "1.1.1-beta.123-beta.456" | sed 's/-beta\.[0-9]*//g'
# 결과: 1.1.1 ✅

변경 파일

  • .github/workflows/release-develop.yml - 워크플로우 수정
  • package.json - 버전 1.1.1로 복구
  • packages/*/package.json - 서브패키지 버전 복구
  • .changeset/fix-beta-workflow-v3.md - changeset 추가

체크리스트

  • step ID 추가
  • 베타 중복 제거 로직 개선
  • 릴리스 순서 변경
  • 태그명 표준화
  • 로컬 테스트 완료

이번에는 반드시 성공하도록 신중하게 수정했습니다.

Seungwoo321 and others added 2 commits June 19, 2025 14:04
🚨 핵심 수정사항:
1. step ID 추가: 'id: version' (누락으로 인한 태그명 오류 해결)
2. 베타 중복 방지 개선: sed 's/-beta\.[0-9]*//g' (모든 베타 제거)
3. 순서 변경: GitHub Release → npm 퍼블리시 (실패시 롤백 가능)
4. 태그명 표준화: vue-pivottable@버전 형식

🔧 버전 복구:
- vue-pivottable: 1.1.1
- @vue-pivottable/plotly-renderer: 2.0.0
- @vue-pivottable/lazy-table-renderer: 1.1.0

📝 변경된 파일:
- .github/workflows/release-develop.yml
- 각 패키지의 package.json (버전 복구)
- changeset 추가

🎯 목표:
- 빈 태그 'v' 생성 방지
- 베타 중복 접미사 방지 (beta.123-beta.456)
- GitHub Release 실패시 npm 퍼블리시 차단

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>
- 작업 전 지침 일치성 자가 점검 프로세스 추가
- 추가 요청 시 sequential-thinking을 활용한 작업 계획 재검토 프로세스 추가
- 핵심 원칙 섹션에 자연스럽게 통합

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>
@Seungwoo321 Seungwoo321 merged commit 1d29e50 into develop Jun 19, 2025
2 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.

2 participants