[openstreetmap/openstreetmap-website] Selected rubocop changes (ruby for good) (#1987)

Andy Allan notifications at github.com
Mon Sep 10 06:21:51 UTC 2018


This PR pulls in three of the rubocop-related changes from the Ruby For Good refactoring PR (#1918). One of them was already fixed in the meantime.
You can view, comment on, or merge this pull request online at:

  https://github.com/openstreetmap/openstreetmap-website/pull/1987

-- Commit Summary --

  * Resolve 96 Rubocop Lint/AmbiguousRegexpLiteral conflicts
  * Remove Lint/ShadowingOuterLocalVariable - already fixed
  * Resolve 34 Rubocop Lint/AmbiguousOperator conflicts

-- File Changes --

    M .rubocop_todo.yml (17)
    M app/helpers/notifier_helper.rb (2)
    M test/controllers/amf_controller_test.rb (62)
    M test/controllers/changeset_controller_test.rb (12)
    M test/controllers/diary_entry_controller_test.rb (4)
    M test/controllers/messages_controller_test.rb (4)
    M test/controllers/old_node_controller_test.rb (2)
    M test/controllers/redactions_controller_test.rb (2)
    M test/controllers/swf_controller_test.rb (6)
    M test/controllers/traces_controller_test.rb (2)
    M test/controllers/user_controller_test.rb (46)
    M test/integration/user_creation_test.rb (2)
    M test/lib/bounding_box_test.rb (4)
    M test/lib/country_test.rb (6)
    M test/models/changeset_test.rb (12)
    M test/models/message_test.rb (4)
    M test/models/node_test.rb (38)
    M test/models/old_node_test.rb (4)
    M test/models/relation_test.rb (22)
    M test/models/tracepoint_test.rb (4)
    M test/models/way_test.rb (22)

-- Patch Links --

https://github.com/openstreetmap/openstreetmap-website/pull/1987.patch
https://github.com/openstreetmap/openstreetmap-website/pull/1987.diff

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/1987
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20180909/4801922d/attachment-0001.html>


More information about the rails-dev mailing list