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

"X" 発言に Twitter スタンプを送る #897

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

Sqrt10-31622776
Copy link
Contributor

@Sqrt10-31622776 Sqrt10-31622776 commented May 21, 2024

"Twitter" を含むメッセージに対して "X" スタンプが来るのに対抗して,"X"(単語中のものは含まない)を含むメッセージに対して "Twitter" スタンプを送るようにしました
初めて TSG の slackbot をいじるので粗祖をしていたらごめんなさい
デバッグとかしていませんが多分動くと信じてます
screenshot-146

@settyan117
Copy link
Contributor

http-local-fwd をリポジトリ内に clone したことで submodule 扱いになってしまっていそう

@Sqrt10-31622776
Copy link
Contributor Author

消してみました,これで行けますか

Copy link
Member

@hakatashi hakatashi left a comment

Choose a reason for hiding this comment

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

[MAY] 動作確認はしましたか? 可能な限りされていることが望ましいです。動作確認をした場合はスクリーンショットをプルリクの最初のメッセージに貼り付けてください。

[MAY] sushi-botにはユニットテストがあるので、可能な限り変更に対応するテストケースが追加されていることが望ましいです。

Comment on lines +360 to +362
}

{
Copy link
Member

Choose a reason for hiding this comment

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

[SHOULD] インデントがスペースになっています。周りに合わせてください

Copy link
Contributor Author

Choose a reason for hiding this comment

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

動作確認をしました.
ユニットテストを追加しました.

@Sqrt10-31622776 Sqrt10-31622776 marked this pull request as draft September 15, 2024 16:34
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