[openstreetmap/openstreetmap-website] Various Erblint fixes (#2187)
Andy Allan
notifications at github.com
Wed Mar 27 10:22:10 UTC 2019
This PR contains various fixes highlighted by erblint. I've only done part of the string literals fixes today, since there's quite a few more to go, to put it mildly.
You can view, comment on, or merge this pull request online at:
https://github.com/openstreetmap/openstreetmap-website/pull/2187
-- Commit Summary --
* erblint: use hash rocket syntax
* erblint: Add or remove curly braces around hash parameters as required
* Exclude most views from string literal checks
* erblint: prefer double quoted strings
* erblint: avoid parentheses for method calls with no arguments
-- File Changes --
M .erb-lint.yml (25)
M app/views/browse/_common_details.html.erb (2)
M app/views/browse/_node.html.erb (2)
M app/views/browse/_relation_member.html.erb (2)
M app/views/browse/_way.html.erb (4)
M app/views/browse/changeset.html.erb (6)
M app/views/diary_entries/comments.html.erb (4)
M app/views/diary_entries/index.html.erb (4)
M app/views/issues/_comments.html.erb (2)
M app/views/issues/index.html.erb (2)
M app/views/issues/show.html.erb (6)
M app/views/layouts/_head.html.erb (18)
M app/views/layouts/_search.html.erb (8)
M app/views/layouts/notifier.html.erb (2)
M app/views/messages/_message_summary.html.erb (6)
M app/views/messages/_sent_message_summary.html.erb (2)
M app/views/notes/_notes_paging_nav.html.erb (4)
M app/views/notifier/_message_body.html.erb (10)
M app/views/reports/new.html.erb (2)
M app/views/site/about.html.erb (8)
M app/views/site/copyright.html.erb (38)
M app/views/site/edit.html.erb (10)
M app/views/site/export.html.erb (44)
M app/views/site/fixthemap.html.erb (2)
M app/views/site/help.html.erb (2)
M app/views/site/id.html.erb (4)
M app/views/site/index.html.erb (2)
M app/views/site/key.html.erb (2)
M app/views/site/offline.html.erb (4)
M app/views/traces/_trace.html.erb (2)
M app/views/traces/_trace_paging_nav.html.erb (4)
M app/views/traces/show.html.erb (4)
M app/views/user_blocks/_blocks.html.erb (4)
M app/views/user_blocks/show.html.erb (2)
M app/views/users/account.html.erb (4)
M app/views/users/login.html.erb (4)
M app/views/users/lost_password.html.erb (2)
M app/views/users/new.html.erb (14)
M app/views/users/reset_password.html.erb (4)
-- Patch Links --
https://github.com/openstreetmap/openstreetmap-website/pull/2187.patch
https://github.com/openstreetmap/openstreetmap-website/pull/2187.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/2187
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20190327/d14d81d0/attachment.html>
More information about the rails-dev
mailing list