Skip to content

Conversation

@uni-kakurenbo
Copy link
Contributor

No description provided.

@uni-kakurenbo uni-kakurenbo requested a review from Pugma September 28, 2025 05:08
@uni-kakurenbo uni-kakurenbo force-pushed the chore/spec-tags-when-releasing branch from 0e1c933 to 00cf401 Compare September 28, 2025 05:09
Copy link
Contributor

@Pugma Pugma left a comment

Choose a reason for hiding this comment

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

作業ありがとうございます!
一点だけ、確認をお願いします…!

git config user.name "GitHub Actions"
git config user.email "actions@github.com"
npm version $NEW_VERSION -m "CI: update to $NEW_VERSION [ci skip]"
Copy link
Contributor

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.

いつも shell-check に怒られるので手癖で足してしまいました
(コミットメッセージの部分は気づいてこっそり force-push したんですが間に合いませんでした.今は直ってると思います、すみません)

@uni-kakurenbo uni-kakurenbo requested a review from Pugma September 28, 2025 05:19
Copy link
Contributor

@Pugma Pugma left a comment

Choose a reason for hiding this comment

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

よさそうです!
ありがとうございます!!!

@uni-kakurenbo uni-kakurenbo merged commit 9b590a3 into master Sep 28, 2025
3 of 4 checks passed
@uni-kakurenbo uni-kakurenbo deleted the chore/spec-tags-when-releasing branch September 28, 2025 05:20
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.

3 participants