Skip to content

Conversation

@yceffort-naver
Copy link
Contributor

Summary

  • 외부 fork에서 오는 PR에 대해 changeset 생성 방법을 안내하는 코멘트 자동 추가
  • 기존 내부 PR의 자동 changeset 감지 기능은 그대로 유지

변경 내용

PR 유형 동작
내부 PR 기존대로 자동 changeset 감지 및 생성
Fork PR 한글/영어로 changeset 생성 방법 안내 코멘트 추가

Fork PR에 달리는 코멘트 예시

## Changeset Guide for External Contributors

**한국어** | [English](#english)

### 한국어
외부 기여자분께 감사드립니다! 🎉
패키지에 변경사항이 있는 경우, changeset 파일을 수동으로 생성해주세요.
...

### English
Thank you for your contribution! 🎉
If your PR includes package changes, please create a changeset file manually.
...

배경

  • 외부 fork 사용자는 ACTION_TOKEN 시크릿에 접근할 수 없어 자동 changeset 감지가 실패함
  • 이를 해결하기 위해 fork PR에는 수동 changeset 생성 방법을 안내하는 코멘트를 자동으로 추가

@yceffort-naver yceffort-naver requested a review from a team as a code owner December 30, 2025 06:23
@npayfebot
Copy link
Collaborator

npayfebot commented Dec 30, 2025

🔍 No files have been changed

Latest commit: 520af8a

Please check your commit.

If packages_dir is not specified or only markdown files are in the changes, detection may fail.

powered by: naverpay changeset detect-add actions

@npayfebot
Copy link
Collaborator

npayfebot commented Dec 30, 2025

🔍 No files have been changed

Latest commit: 520af8a

Please check your commit.

powered by: naverpay size-action

2-one-week
2-one-week previously approved these changes Dec 30, 2025
Copy link
Member

@2-one-week 2-one-week left a comment

Choose a reason for hiding this comment

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

LGTM

- changeset-actions의 fork_guide_enabled 옵션 사용
- 내부 PR: 기존대로 자동 changeset 감지 및 생성
- Fork PR: 한글/영어로 changeset 생성 방법 안내 코멘트 추가
@yceffort-naver
Copy link
Contributor Author

@2-one-week 님 요고 한번만 다시 승인해주셔도 되나요?

Copy link

@boxersb boxersb left a comment

Choose a reason for hiding this comment

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

Looks Good To Me!

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.

5 participants