diff options
author | Yamagishi Kazutoshi <ykzts@desire.sh> | 2019-12-03 02:25:43 +0900 |
---|---|---|
committer | multiple creatures <dev@multiple-creature.party> | 2020-02-21 04:03:46 -0600 |
commit | 28073bbcd6010de077e528f0447b35e51905ed23 (patch) | |
tree | d4751866764c114f4a3030aae835922c6b8a0570 | |
parent | 924c2c0880a96d8e502427e0b697148916c021f1 (diff) |
port tootsuite#12534 to monsterfork: Update ESLint and RuboCop in Code Climate
-rw-r--r-- | .codeclimate.yml | 4 | ||||
-rw-r--r-- | .rubocop.yml | 6 | ||||
-rw-r--r-- | app/models/media_attachment.rb | 2 | ||||
-rw-r--r-- | lib/paperclip/lazy_thumbnail.rb | 4 |
4 files changed, 11 insertions, 5 deletions
diff --git a/.codeclimate.yml b/.codeclimate.yml index 571507a54..9817d7f1c 100644 --- a/.codeclimate.yml +++ b/.codeclimate.yml @@ -27,10 +27,10 @@ plugins: enabled: true eslint: enabled: true - channel: eslint-5 + channel: eslint-6 rubocop: enabled: true - channel: rubocop-0-71 + channel: rubocop-0-76 sass-lint: enabled: true exclude_patterns: diff --git a/.rubocop.yml b/.rubocop.yml index 8bd4c867f..9a68becbb 100644 --- a/.rubocop.yml +++ b/.rubocop.yml @@ -71,6 +71,9 @@ Naming/MemoizedInstanceVariableName: Rails: Enabled: true +Rails/EnumHash: + Enabled: false + Rails/HasAndBelongsToMany: Enabled: false @@ -102,6 +105,9 @@ Style/Documentation: Style/DoubleNegation: Enabled: true +Style/FormatStringToken: + Enabled: false + Style/FrozenStringLiteralComment: Enabled: true diff --git a/app/models/media_attachment.rb b/app/models/media_attachment.rb index 502ebb0b5..36282d010 100644 --- a/app/models/media_attachment.rb +++ b/app/models/media_attachment.rb @@ -325,7 +325,7 @@ class MediaAttachment < ApplicationRecord width: width, height: height, size: "#{width}x#{height}", - aspect: width.to_f / height.to_f, + aspect: width.to_f / height, } end diff --git a/lib/paperclip/lazy_thumbnail.rb b/lib/paperclip/lazy_thumbnail.rb index 5992e76d8..597d7f0bc 100644 --- a/lib/paperclip/lazy_thumbnail.rb +++ b/lib/paperclip/lazy_thumbnail.rb @@ -9,8 +9,8 @@ module Paperclip min_side = [@current_geometry.width, @current_geometry.height].min.to_i options[:geometry] = "#{min_side}x#{min_side}#" if @target_geometry.square? && min_side < @target_geometry.width elsif options[:pixels] - width = Math.sqrt(options[:pixels] * (@current_geometry.width.to_f / @current_geometry.height.to_f)).round.to_i - height = Math.sqrt(options[:pixels] * (@current_geometry.height.to_f / @current_geometry.width.to_f)).round.to_i + width = Math.sqrt(options[:pixels] * (@current_geometry.width.to_f / @current_geometry.height)).round.to_i + height = Math.sqrt(options[:pixels] * (@current_geometry.height.to_f / @current_geometry.width)).round.to_i options[:geometry] = "#{width}x#{height}>" end |