Closed kachick closed 12 years ago
些細な点ですが、下記について利用方針を統一した方が読みやすくなると感じたため、パッチを送ります。 ご検討下さい
1. require と require_relative の混在。
2. require の引数に渡す文字列リテラルで「"」と「'」の混在。
06b40920755364337a08a49d8fc17b407026532b
1. ->{} 記法を用いている箇所もあることからRuby1.8.nまでを考慮する必要は無いものと類推し、 可能な限り require_relative を使いました。
2. 他の文字列リテラル利用箇所から類推し、「"」へ統一しました。
test/test_map.rb に於いて、 List 定数が読み込まれていない為と思われるエラーが発生しました。 これの解消を図り、 f8b189bdd1aeeb908966ff8b1d6ee49bf2633e53 にて require_relative を増やしました。
提案ありがとうございます。マージしました。
提案
些細な点ですが、下記について利用方針を統一した方が読みやすくなると感じたため、パッチを送ります。 ご検討下さい
1. require と require_relative の混在。
2. require の引数に渡す文字列リテラルで「"」と「'」の混在。
06b40920755364337a08a49d8fc17b407026532b
対応内容
1. ->{} 記法を用いている箇所もあることからRuby1.8.nまでを考慮する必要は無いものと類推し、 可能な限り require_relative を使いました。
2. 他の文字列リテラル利用箇所から類推し、「"」へ統一しました。
注記
test/test_map.rb に於いて、 List 定数が読み込まれていない為と思われるエラーが発生しました。 これの解消を図り、 f8b189bdd1aeeb908966ff8b1d6ee49bf2633e53 にて require_relative を増やしました。