Skip to content

Commit

Permalink
Fix Rubocop offenses
Browse files Browse the repository at this point in the history
  • Loading branch information
monfresh committed Apr 6, 2017
1 parent e9b19e5 commit 5370fb1
Show file tree
Hide file tree
Showing 7 changed files with 6 additions and 11 deletions.
4 changes: 2 additions & 2 deletions .rubocop.yml
Expand Up @@ -195,7 +195,7 @@ Style/TrailingCommaInArguments:
# If `consistent_comma`, the cop requires a comma after the last argument,
# for all parenthesized method calls with arguments.
EnforcedStyleForMultiline: no_comma
SupportedStyles:
SupportedStylesForMultiline:
- comma
- consistent_comma
- no_comma
Expand All @@ -206,7 +206,7 @@ Style/TrailingCommaInLiteral:
# If `consistent_comma`, the cop requires a comma after the last item of all
# non-empty array and hash literals.
EnforcedStyleForMultiline: no_comma
SupportedStyles:
SupportedStylesForMultiline:
- comma
- consistent_comma
- no_comma
1 change: 0 additions & 1 deletion Gemfile
Expand Up @@ -29,7 +29,6 @@ end
# dev and debugging tools
group :development do
gem 'better_errors'
gem 'binding_of_caller', '~> 0.7.2', platforms: [:mri_19, :rbx]
gem 'bummr'
gem 'derailed'
gem 'flamegraph'
Expand Down
4 changes: 0 additions & 4 deletions Gemfile.lock
Expand Up @@ -57,8 +57,6 @@ GEM
coderay (>= 1.0.0)
erubis (>= 2.6.6)
rack (>= 0.9.0)
binding_of_caller (0.7.2)
debug_inspector (>= 0.0.1)
builder (3.2.3)
bummr (0.2.0)
rainbow
Expand Down Expand Up @@ -102,7 +100,6 @@ GEM
tins (~> 1.6)
crack (0.4.3)
safe_yaml (~> 1.0.0)
debug_inspector (0.0.2)
declarative (0.0.9)
declarative-option (0.1.0)
derailed (0.1.0)
Expand Down Expand Up @@ -387,7 +384,6 @@ PLATFORMS
DEPENDENCIES
actionmailer
better_errors
binding_of_caller (~> 0.7.2)
bummr
capybara
compass-rails
Expand Down
2 changes: 1 addition & 1 deletion app/helpers/detail_format_helper.rb
Expand Up @@ -2,7 +2,7 @@ module DetailFormatHelper
# List of fields that determine whether or not to show the
# Contact section in the details view
def location_contact_fields
[:website, :email, :phones]
%i(website email phones)
end

# Strips http:// or https:// from URL
Expand Down
2 changes: 1 addition & 1 deletion app/helpers/result_summary_helper.rb
Expand Up @@ -21,7 +21,7 @@ def map_summary
def search_results_page_title
search_terms = request.query_parameters.
except(:utf8, :service_area).
map { |k, v| "#{k}: #{v}" unless v.blank? }.
map { |k, v| "#{k}: #{v}" if v.present? }.
compact.join(', ')
"Search results for: #{search_terms}"
end
Expand Down
2 changes: 1 addition & 1 deletion config/application.rb
Expand Up @@ -5,7 +5,7 @@
require 'action_mailer/railtie'
require 'sprockets/railtie'

SETTINGS ||= YAML.load(File.read(File.expand_path('../settings.yml', __FILE__)))
SETTINGS ||= YAML.safe_load(File.read(File.expand_path('../settings.yml', __FILE__)))
SETTINGS.merge! SETTINGS.fetch(Rails.env, {})
SETTINGS.symbolize_keys!

Expand Down
2 changes: 1 addition & 1 deletion spec/rails_helper.rb
Expand Up @@ -18,7 +18,7 @@

# Requires supporting ruby files with custom matchers and macros, etc,
# in spec/support/ and its subdirectories.
Dir[Rails.root.join('spec/support/**/*.rb')].each { |f| require f }
Dir[Rails.root.join('spec', 'support', '**', '*.rb')].each { |f| require f }

RSpec.configure do |config|
# Helper methods for use with Capybara feature specs.
Expand Down

0 comments on commit 5370fb1

Please sign in to comment.