Skip to content

Conversation

@h-east
Copy link
Member

@h-east h-east commented Oct 6, 2024

No description provided.

Copy link
Contributor

@tsuyoshicho tsuyoshicho left a comment

Choose a reason for hiding this comment

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

これくらいかな?

Co-authored-by: Tsuyoshi CHO <[email protected]>
Copy link
Contributor

@tsuyoshicho tsuyoshicho left a comment

Choose a reason for hiding this comment

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

"な" を次に送らなくてもよかったかもですが、approve

@h-east
Copy link
Member Author

h-east commented Oct 6, 2024

"な" を次に送らなくてもよかったかもですが、approve

送って正解でしたよ。

@tsuyoshicho
Copy link
Contributor

"な" を次に送らなくてもよかったかもですが、approve
送って正解でしたよ。

GitHubの差分表示だと、桁把握ムズイ...
よかったよかった

Copy link
Member

@mityu mityu left a comment

Choose a reason for hiding this comment

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

少し気づいた点をコメントしていますが、要修正と思うところはなかったのでレビューとしては approve とします。

@h-east h-east merged commit b44d0f2 into master Oct 7, 2024
3 checks passed
@h-east h-east deleted the hh-update-options branch October 7, 2024 06:12
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.

4 participants