Bundle Update on 2017-04-07

[deppbot]

Updated RubyGems:

• Updated: faraday, 0.11.0...0.12.0.1 (CHANGELOG)
• Updated: rubocop, 0.48.0...0.48.1 (CHANGELOG)

--

Brought to you by deppbot - Automated Security and Dependency Updates
2 Pull Requests sent and 1 minute, and 21 seconds saved so far.

_{Feedback or Bug Reports? Please let us know via Email or GitHub Issues.}

[colorbox]

rubocop激おこ

app/controllers/order_items_controller.rb:4:40: C: Style/PercentLiteralDelimiters: %i-literals should be delimited by [ and ].
  before_action :set_order_item, only: %i(edit update destroy receive)
                                       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

[colorbox]

rubocopのバージョン上がったせいのようですね

[colorbox]

0.48.1で加わったこれが原因みたい。
rubocop/rubocop#4039
ruby style guideに準拠させたとのこと

[colorbox]

なるほど
rubocop/rubocop#4231
rubocop/ruby-style-guide#526

[colorbox]

うーん、流石にこの書き方に文句言われるのはきっついな

Offenses:

app/models/order.rb:18:3: W: Lint/AmbiguousBlockAssociation: Parenthesize the param `-> {
    where('date >= ?', Time.current) \
      .limit(SHOW_COUNT) \
      .order(date: :asc) \
      .reject(&:closed?)
  } to make sure that the block will be associated with the ->` method call.
  scope :available, -> { ...
  ^^^^^^^^^^^^^^^^^^^^^^

[colorbox]

こう書くと文句は言われないけど、不要な丸括弧を追加している感じで非常に嫌。

  scope :available, (-> {
    where('date >= ?', Time.current) \
      .limit(SHOW_COUNT) \
      .order(date: :asc) \
      .reject(&:closed?)
  }

[colorbox]

rubocop/rubocop#4237

[colorbox]

バグとして報告されているけど、rubocopの最新版に取り込まれていないのか。
このPRはcloseして、次のBundleUpdateを待つほうが良さそう。

[colorbox]

一旦closeして次のbundle updateを待ちます。