Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[공통] changeset action 을 npay changeset action 으로 이사 #104

Merged
merged 3 commits into from
Sep 26, 2024

Conversation

npayolaf
Copy link
Contributor

Related Issue

Describe your changes

  • npay 의 detect add, publish 를 이용하도록 변경합니다

Request

@npayolaf npayolaf marked this pull request as ready for review September 26, 2024 09:57
@npayolaf npayolaf requested a review from a team as a code owner September 26, 2024 09:57
@NaverPayDev NaverPayDev deleted a comment from changeset-bot bot Sep 26, 2024
@npayolaf npayolaf self-assigned this Sep 26, 2024
id: changesets
uses: changesets/action@v1
uses: NaverPayDev/changeset-actions/actions/publish@main
Copy link
Member

Choose a reason for hiding this comment

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

사소한 의견이긴한데,

chageset-actions 라는 레포 이름이 action이라는걸 내포하고 있으니,,

NaverPayDev/changeset-actions/publish@main

는 어떠신가용??

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Copy link
Contributor Author

Choose a reason for hiding this comment

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

변경완 !

@npayolaf npayolaf requested a review from 2-one-week September 26, 2024 10:23
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.

의견반영 감사드립니다!!! 고생 많으셨습니다~~

@npayolaf npayolaf merged commit 0eff9ca into main Sep 26, 2024
3 checks passed
@npayolaf npayolaf deleted the repo/103 branch September 26, 2024 11:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants