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

Add output Github #175

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

Jankabi
Copy link

@Jankabi Jankabi commented Aug 8, 2024

  • 7という数字がつくときはGitHubを表示
    を実装しました。レビューお願いいたします。

@Jankabi
Copy link
Author

Jankabi commented Aug 8, 2024

@Jankabi
インデントがおかしいようです。

@Jankabi
Copy link
Author

Jankabi commented Aug 8, 2024

@Jankabi
今回の実装に関するテストコードがないので、追加してください。

@Jankabi
Copy link
Author

Jankabi commented Aug 8, 2024

仕様の質問があります。
例えば、75という数字は本件の実装の前は"fizzbuzz"と表示されていたかと思います。
保険の実装後だと"7という数字"に該当するので"GitHub"と表示してしまいます。
"fizzbuzzGitHub"などのように、複合して表示したいということはありませんか?

@Jankabi
Copy link
Author

Jankabi commented Aug 8, 2024

なるほど。そういったケースについて触れていませんでした。
仕様を以下の通りとしてください。

  • 3の倍数や5の倍数でも、7という数字がつくときはGitHubを表示

@Jankabi
Copy link
Author

Jankabi commented Aug 8, 2024

テストコードを追加して、実装が完了しました。
レビュー後に問題なければマージをお願いします。

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.

1 participant