Skip to content

コミットメッセージの複数選択をTUIで可能にする #14

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

Merged
merged 13 commits into from
Sep 3, 2023

Conversation

cocoide
Copy link
Owner

@cocoide cocoide commented Sep 2, 2023

#2

@cocoide cocoide requested a review from mochi-yu September 2, 2023 13:46
@cocoide cocoide changed the title Feature/message select コミットメッセージの複数選択をTUIで可能にする #2 Sep 2, 2023
@cocoide cocoide changed the title コミットメッセージの複数選択をTUIで可能にする #2 コミットメッセージの複数選択をTUIで可能にする Sep 2, 2023
@cocoide
Copy link
Owner Author

cocoide commented Sep 2, 2023

@mochi-yu
すまん、あっちのブランチ間違えて消してしまったからこっちに書く。クラスの名前の意図を伝えるとあえて抽象的にしたのは他にもMessage系のServiceを作る可能性があってその度Service作るの面倒くさいなと思ったから。
命名はすぐ変えれるし最初はシンプルでいいかなと思った。
GenerateMessageServiceとかでもいいのかなと思ってる。

@mochi-yu
Copy link
Collaborator

mochi-yu commented Sep 3, 2023

@cocoide コンフリクトが起きてるので、そこの対処はできる?
自分の方でコンフリクト解消しました🙇‍♂️

@mochi-yu mochi-yu self-requested a review September 3, 2023 09:06
@cocoide cocoide merged commit fe52414 into main Sep 3, 2023
@mochi-yu mochi-yu deleted the feature/message-select branch September 3, 2023 09:39
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.

2 participants