Skip to content

Add output Github#175

Open
Jankabi wants to merge 3 commits into
github-book:masterfrom
Jankabi:7-case-output-github
Open

Add output Github#175
Jankabi wants to merge 3 commits into
github-book:masterfrom
Jankabi:7-case-output-github

Conversation

@Jankabi

@Jankabi Jankabi commented Aug 8, 2024

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

@Jankabi

Jankabi commented Aug 8, 2024

Copy link
Copy Markdown
Author

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

@Jankabi

Jankabi commented Aug 8, 2024

Copy link
Copy Markdown
Author

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

@Jankabi

Jankabi commented Aug 8, 2024

Copy link
Copy Markdown
Author

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

@Jankabi

Jankabi commented Aug 8, 2024

Copy link
Copy Markdown
Author

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

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

@Jankabi

Jankabi commented Aug 8, 2024

Copy link
Copy Markdown
Author

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

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