Closed shimoju closed 6 years ago
rubocopに捕まった…
Metrics/CyclomaticComplexity
:このエラー処理の分岐が増えるのはどうしようもないのでExcludeにした。ほかのところでも引っかかるなら10くらいに緩めるかもMetrics/MethodLength
:10はさすがに厳しすぎるでしょってことで20にしたOffenses:
lib/metabase/error.rb:5:5: C: Metrics/CyclomaticComplexity: Cyclomatic complexity for from_response is too high. [10/6]
def self.from_response(response) ...
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
lib/metabase/error.rb:5:5: C: Metrics/MethodLength: Method has too many lines. [13/10]
def self.from_response(response) ...
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
Metrics/CyclomaticComplexity:
Exclude:
- lib/metabase/error.rb
Metrics/MethodLength:
Max: 20
各HTTPステータスを表したエラークラスを追加しました。 とりあえず明らかに取りうるステータスだけ入れています
その他の400、500系ステータスがきたらClientError、ServerErrorでキャッチされます(これもoctokit参考) https://github.com/octokit/octokit.rb/blob/master/lib/octokit/error.rb