code-your-ruby

code-your-ruby copied to clipboard

ある一定のルールで運行している電車の時刻表を算出する機能

6

## このPRについて [こちらの仕様](https://github.com/chooyan-eng/code-your-ruby/blob/master/specification/timetable.md)の処理を実装しました ## 実行結果 ```sh cd user/h5y1m141 ruby ./main.rb ``` 上記実行結果は以下のように表示されます ``` 各駅停車の時刻表 ["11:10", "11:14", "11:18", "11:23", "11:27", "11:31", "11:35", "11:39", "11:43", "11:47", "11:51", "11:55", "11:59", "12:03", "12:07", "12:11",...

h5y1m141

レビュワーがレビューしやすくなるよう、レビューの方針を明確にする

5

@ttanimichi さんより、 > どこまで深くレビューすべきなのかなっていうのがふわっとしてて。あまり表面的なレビューだけしても価値が薄いでしょうし とコメントいただいた通り、現状ではどのレベルでレビューをすればよいかの指針がないことによってレビューしづらい（何からツッコめばよいか分からない）状況になっています。 また、深いレベルでレビューしようとすると、それだけレビュワーの負担ばかりが増えてしまう問題、GitHub上でのやりとりでは限界がある点も課題となっています。 レビュワーがレビューしやすいために、何をどうすれば良いかを考えたいです。

chooyan-eng

FizzBuzz実装しました

1

Ruby on Railsチュートリアルをぼちぼち進めている身ですがやってみました。 「任意の整数を入力として与えた場合」という要件だったので整数以外は考慮していません。 また0や負数も整数として考慮し、テストを書きました。

nyoronjp

Implement fizzbuzz

1

ﾚﾋﾞｭｰ ｵﾈｶﾞｲｼﾏｽ!! 🙏 # 概要 FizzBuzz の実装をしました * 例外規定などの記述が見られないので、 [fizz_buzz.md](https://github.com/chooyan-eng/code-your-ruby/blob/1510dd2aa5c3f27a0fd5c01574a08e9dbf93c02e/specification/fizz_buzz.md) 1510dd2aa5c3f27a0fd5c01574a08e9dbf93c02e の要件に沿うよう素朴に実装してみました * RSpec によるテストコードの記述を実施; FizzBuzz 変換対象のものとそうでないもの、それぞれを配列に列挙(根拠なく1から100まで) * FizzBuzz module に detect メソッドを定義し FizzBuzz の変換を実施 ## その他 * `if ......

gouf

想定するRubyのバージョン記載

3

## このIssueについて @chooyan-eng さん、興味あって早速リポジトリをフォークしてtimetableのお題についてとりかかろうと思って作業してて気になったのが、このリポジトリでのRubyの想定バージョンについて記載がありませんでした。 > 自分ひとりでの勉強では得られない視点や技術を身に着けることができます。 という観点に加えて - 実際のお仕事だとRubyのバージョンによる差異でおもいがけない挙動でハマる - Rubyのバージョンが新しくなると、割りとカッコいい書き方とかも使えるようになる - とはいえ、自分が最近やってるお仕事だと最新に追従出来てないですが 😅 という点も踏まえると想定するRubyのバージョン記載したほうが良いかなと思うのですがいかがでしょうか？

h5y1m141