-
Notifications
You must be signed in to change notification settings - Fork 309
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
ソング関連のe2eテストを追加 #1849
ソング関連のe2eテストを追加 #1849
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
1点だけコメントしましたが、あってもなくてもいいかもです...!
Co-authored-by: Yuto Ashida <[email protected]>
レビューを反映しました。 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ほぼLGTMです!!
ちょっとこっちで書いてみてpushしてみるので @weweweok さんにそれ見てもらえると助かります!!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTMです!!
解説コメント書いてみました!
@weweweok 見ていただけると・・・!!
ありがとうございます。 細かいですが、関数を移動してみました。ご確認いただけると🙇♂ |
ありがとうございます! マージします!! e2eテスト追加とても助かります。コードも良い感じだったと思います! |
内容
ソング関連のe2eテストを追加しました。
テストケース
関連 Issue