다시 릴리즈 PR 사용하도록 Tagging 워크플로우 변경 #727
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
DaleSeo
force-pushed
the
725-release-pr
branch
from
67f901b to
3868d15
Compare
DaleSeo
force-pushed
the
725-release-pr
branch
from
3868d15 to
63fdaf5
Compare
Bundle ReportBundle size has no change ✅ |
DaleSeo
force-pushed
the
725-release-pr
branch
from
63fdaf5 to
b36cb78
Compare
- release-pr.yml: 버전 bump PR 생성 (수동 트리거) - tagging.yml: release/ 브랜치 PR 머지 시 자동 태그 생성 및 릴리스 발행
DaleSeo
force-pushed
the
725-release-pr
branch
from
b36cb78 to
bcd2fe0
Compare
hyoseong1994
reviewed
Jan 7, 2026
Contributor
hyoseong1994
left a comment
hyoseong1994
left a comment
There was a problem hiding this comment.
pr 자동화 방법에 대해 고민중이었는데 이렇게 바로 작업해주시다니 감사합니다!
.github/workflows/release-pr.yml
Outdated
Comment on lines
29
to
30
| - name: Bump version | ||
| run: npm version ${{ inputs.bump }} --no-git-tag-version |
Contributor
There was a problem hiding this comment.
bump version가 main에서 이루어지는거같은데 그러면 origin/main 이랑 달라질것 같습니다.
스크립트 중에 git switch -c "$BRANCH" origin/main 이 부분 이후에 bump version을 진행해야 정상동작 할 것으로 예상됩니다.
Contributor
There was a problem hiding this comment.
예상과 다르게 정상동작하는거 같네요 신기하네요
Member
Author
There was a problem hiding this comment.
@hyoseong1994 님, --no-git-tag-version 옵션은 npm version 명령어가 기본적으로 하는 커밋과 태그 생성을 모두 건너뛰고, package.json 내의 버전만 수정합니다. 즉, 파일 변경사항이 Working directory에 남아있어요. Git은 브랜치를 바꿔도 Stage 영역으로 넘어오지 않는 변경사항을 버리지 않거든요. 아무래도 옵션 이름 때문에 오해하기 쉬울 것 같아서 주석을 추가해놓겠습니다.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
변경 사항
PR #718 병합 후에 태깅을 시도했는데 워크플로우가 실패하였습니다.
https://github.com/DaleStudy/daleui/actions/runs/20702448321
PR 병합 전 피쳐 브랜치 상대로 테스트할 때는 잘 됐던 워크플로우가 실패한 이유는 main 브랜치에 직접 push를 금지하는 저장소의 브랜츠 보호 규칙을 위반하기 때문입니다.
이 문제를 해결하기 위해서 패키지 버전을 올리고 Release PR을 자동으로 생성하기 위한
release-pr.yml워크플로우를 추가하고, 기존tagging.yml플로우는 릴리즈 PR 머지 시 자동으로 수행되고 태그만 push하도록 수정하였습니다.목적
이렇게 워크플로우를 분리하게 되면, 패키지 버전 변경도 다른 변경처럼 PR 병합을 통해 main 브랜치로 병합될 수 있어서 저장소의 브랜츠 보호 규칙을 준수하게 됩니다. 여전히 패키지 변경과 GitHub Actions 내에서 수행되기 때문에 개발자의 실수를 최소화합니다. 릴리즈 PR을 생성하고 리뷰해야한다는 부담이 생기지만, 이 것은 보는 시각에 따라서 장점이 될 수 있을 것입니다. 오히려 릴리즈 PR없이 패키지 발행을 하는 예전 프로세스가 지나치게 자동화를 하여 관행이 어긋나는 느낌이 있었습니다. PR을 통해서 릴리즈 내용을 검토할 수 있으니 배포 프로세스가 좀 더 안정화가 되면 Draft Release 단계를 생략하는 것을 나중에 고려해볼 수 있을 것 같네요.
리뷰어에게
수동으로 테스트를 진행하여 새로운 워크플로우가 의도대로 작동함을 확인하였습니다.
PR 작성자 체크 리스트
UI 변경 없음