rubocop-govuk 3.3.1 → 3.7.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 86994e90da9017502338fcb8274d44d4fd5843c9e346d4e2b0555c559cd52eaf
-  data.tar.gz: 32a0cd421dace4d0c1fd0ff577d3a21f5e5b8a5b395f283af0941d68bb8c5c7c
+  metadata.gz: 9460cb1bd124d0db3bad61e69a4fca8fea76d1b58197766820aca957f4241854
+  data.tar.gz: 13319b8ceece3b49c2c4363f23c418cebd5f87db081fe1eecb63afb777c88194
 SHA512:
-  metadata.gz: 2f1e4b9c0ddd63aa97de8097d3e5f7549d2b311a18573a3d9b27414e6a3df6b25c60c1e2cd103d92c352f5f6c52671dc08bdc7b226971f0675700f1847a0aaed
-  data.tar.gz: 460004238e8fe1a6c499675b9f909b31d646d0672506651d6eb8bfb9c5c53e6def71aabc1e94ada7f8fc764603477440ac3bcb89a918e0792e0b597d02b08ab9
+  metadata.gz: 981c47d318c535003c5e3b5206ca85ead3fda8895c770d4957cc4ee19f698d4bd04ad3483c17d064b079b87e9276c188ce06f478413dc2ed11ee8b2ec2b61c7c
+  data.tar.gz: 127e187db28bad0c6803c1fb8e8208277d55617698aef72befcddb4285449a62a65ee91836808b252f2a75c78fb40699d7b66ab0d4804cb523306d281866838a

data/CHANGELOG.md

@@ -1,3 +1,25 @@
+# 3.7.0
+
+* Turn a load of Cops back on (#52)
+
+# 3.6.0
+
+* Remove config that matches RuboCop defaults (#47)
+* Reorganise all the Cops (#44)
+
+# 3.5.0
+
+* Disable pending cops by default (#37)
+
+# 3.4.0
+
+* Add rubocop-rake cops (#32)
+* Revert #27 (hash transform cops) (#31)
+
+# 3.3.2
+
+* Exclude /tmp directory (#29)
+
 # 3.3.1
 
 * Exclude two unsafe style cops (#27)

data/README.md

@@ -16,7 +16,7 @@ Inherit rules from the gem by adding the following to your project's RuboCop con
 ```yaml
 # .rubocop.yml
 inherit_gem:
-  rubocop-govuk: 
+  rubocop-govuk:
     - config/default.yml
 ```
 
@@ -25,14 +25,18 @@ or if you also need Rails specific rules:
 ```yaml
 # .rubocop.yml
 inherit_gem:
-  rubocop-govuk: 
+  rubocop-govuk:
     - config/default.yml
     - config/rails.yml
+
+inherit_mode:
+  merge:
+    - Exclude
 ```
 
 ## Usage
 
-Run RuboCop: 
+Run RuboCop:
 
 ```sh
 bundle exec rubocop

data/config/bundler.yml

@@ -0,0 +1,12 @@
+# Introduced in: 278fc6e903834acfb37565795a196b85282560b0
+# "A common pattern in our Gemfiles is to declare a gem
+# differently if we're in a dev mode or not. For example
+#
+# if ENV['API_DEV']
+#   gem 'gds-api-adapters', path: '../gds-api-adapters'
+# else
+#   gem 'gds-api-adapters', "~> 50.8.0"
+# end"
+# TODO: see if we can remove this (no longer valid??)
+Bundler/DuplicatedGem:
+  Enabled: false

data/config/default.yml

@@ -1,8 +1,11 @@
 AllCops:
+  NewCops: disable
   Exclude:
     - 'bin/**'
     - 'config.ru'
-    - 'tmp/**'
+    - 'tmp/**/*'
+    - 'db/schema.rb'
+    - 'db/migrate/201*'
 
   DisplayCopNames:
     Description: 'Display cop names in offense messages'
@@ -17,8 +20,10 @@ AllCops:
     Enabled: true
 
 inherit_from:
-  - gds-ruby-styleguide.yml
-  - other-lint.yml
-  - other-style.yml
-  - other-metrics.yml
-  - other-excludes.yml
+  - bundler.yml
+  - layout.yml
+  - lint.yml
+  - metrics.yml
+  - naming.yml
+  - style.yml
+  - rake.yml

data/config/layout.yml

@@ -0,0 +1,51 @@
+# Part of the orignal GDS styleguide
+# "Use empty lines between defs and to break up a method into logical paragraphs."
+# https://github.com/alphagov/styleguides/blob/6395a10d41c3938f4c147cda443fd83f854c3e7a/ruby.md#general
+Layout/EmptyLineBetweenDefs:
+  Description: Use empty lines between defs.
+  Enabled: true
+  AllowAdjacentOneLineDefs: false
+
+# https://github.com/alphagov/govuk-lint/pull/7
+# "There are occasions where following this rule forces you to make the
+# code less readable. This is particularly the case for tests where method
+# names form the test descriptions. Keeping test descriptions under a
+# certain length can force them to become cryptic."
+Layout/LineLength:
+  Description: Limit lines to 80 characters.
+  Enabled: false
+
+# TODO: unclear why this is here!
+# https://github.com/alphagov/govuk-lint/commit/5e42ec5690e3d4cc8fff44d0b0268c8e581c423a
+Layout/MultilineMethodCallIndentation:
+  Enabled: false
+
+# Part of the orignal GDS styleguide
+# "Outdent private etc"
+# https://github.com/alphagov/styleguides/blob/6395a10d41c3938f4c147cda443fd83f854c3e7a/ruby.md#classes
+Layout/AccessModifierIndentation:
+  Description: Check indentation of private/protected visibility modifiers.
+  Enabled: true
+  EnforcedStyle: outdent
+
+# Introduced in: c69a7eb3af955d6c4c0cf0c3cec8e9f330c74429
+# TODO: unclear why this is here! Suggest enabling this with
+# "EnforcedStyle: consistent".
+Layout/FirstArrayElementIndentation:
+  Description: >-
+    Checks the indentation of the first element in an array
+    literal.
+  Enabled: false
+
+# Introduced in: c69a7eb3af955d6c4c0cf0c3cec8e9f330c74429
+# TODO: unclear why this is here! Suggest enabling this with
+# "EnforcedStyle: consistent".
+Layout/FirstHashElementIndentation:
+  Description: 'Checks the indentation of the first key in a hash literal.'
+  Enabled: false
+
+# Introduced in: 9b2a744ab119d7797aaf423abcec914360f4208e
+# "More lenient on multi-line operations"
+# TODO: unclear why this is here!
+Layout/MultilineOperationIndentation:
+  EnforcedStyle: indented

data/config/lint.yml

@@ -0,0 +1,5 @@
+# Introduced in: c69a7eb3af955d6c4c0cf0c3cec8e9f330c74429
+# TODO: unclear why this is here!
+Lint/FlipFlop:
+  Description: 'Checks for flip flops'
+  Enabled: false

data/config/metrics.yml

@@ -0,0 +1,79 @@
+# Introduced in: 195e1d6934e610efed35151cfe90b01b805d7581
+# Introduced in: b7a98b43a4b0644b8d4ac40aa3ddaf35f06fa58f
+# "Many test frameworks use blocks for the entire test
+# case in a file as well as for individual tests within the file.
+# It's not unusual for a test case to be several lines
+# longer than the upper limit of the BlockLength cop,
+# and there's very little we can do to avoid that so
+# there's no point running this cop on our tests."
+Metrics/BlockLength:
+  Enabled: true
+  Exclude:
+    - test/**/*
+    - "**/spec/**/*"
+  ExcludedMethods: ["namespace"]
+
+# Introduced in: 736b3d295f88b9ba6676fc168b823535582388c2
+# "Disable opinionated cops"
+#
+# We should avoid cops that are based on heuristics, since
+# it's not clear what action to take to fix an issue.
+Metrics/AbcSize:
+  Enabled: false
+
+# Introduced in: 736b3d295f88b9ba6676fc168b823535582388c2
+# "Disable opinionated cops"
+#
+# We should avoid cops that are based on heuristics, since
+# it's not clear what action to take to fix an issue.
+Metrics/ClassLength:
+  Enabled: false
+
+# Introduced in: 736b3d295f88b9ba6676fc168b823535582388c2
+# "Disable opinionated cops"
+#
+# We should avoid cops that are based on heuristics, since
+# it's not clear what action to take to fix an issue.
+Metrics/ModuleLength:
+  Enabled: false
+
+# Introduced in: 736b3d295f88b9ba6676fc168b823535582388c2
+# "Disable opinionated cops"
+#
+# We should avoid cops that are based on heuristics, since
+# it's not clear what action to take to fix an issue.
+Metrics/PerceivedComplexity:
+  Enabled: false
+
+# Introduced in: c69a7eb3af955d6c4c0cf0c3cec8e9f330c74429
+# TODO: unclear why this is here!
+Metrics/BlockNesting:
+  Description: 'Avoid excessive block nesting'
+  Enabled: false
+
+# Analog of: 736b3d295f88b9ba6676fc168b823535582388c2
+# "Disable opinionated cops"
+#
+# We should avoid cops that are based on heuristics, since
+# it's not clear what action to take to fix an issue.
+Metrics/CyclomaticComplexity:
+  Description: 'Avoid complex methods.'
+  Enabled: false
+
+# Analog of: 736b3d295f88b9ba6676fc168b823535582388c2
+# "Disable opinionated cops"
+#
+# We should avoid cops that are based on heuristics, since
+# it's not clear what action to take to fix an issue.
+Metrics/MethodLength:
+  Description: 'Avoid methods longer than 10 lines of code.'
+  Enabled: false
+
+# Analog of: 736b3d295f88b9ba6676fc168b823535582388c2
+# "Disable opinionated cops"
+#
+# We should avoid cops that are based on heuristics, since
+# it's not clear what action to take to fix an issue.
+Metrics/ParameterLists:
+  Description: 'Avoid parameter lists longer than three or four parameters.'
+  Enabled: false

data/config/naming.yml

@@ -0,0 +1,25 @@
+# Introduced in: c69a7eb3af955d6c4c0cf0c3cec8e9f330c74429
+# TODO: conflicts with the original GDS styleguide
+# "Avoid get/set method names"
+# https://github.com/alphagov/styleguides/blob/6395a10d41c3938f4c147cda443fd83f854c3e7a/ruby.md#naming
+Naming/AccessorMethodName:
+  Description: Check the naming of accessor methods for get_/set_.
+  Enabled: false
+
+# Introduced in: c69a7eb3af955d6c4c0cf0c3cec8e9f330c74429
+# TODO: unclear why this is here!
+Naming/AsciiIdentifiers:
+  Description: 'Use only ascii symbols in identifiers.'
+  Enabled: false
+
+# Introduced in: c69a7eb3af955d6c4c0cf0c3cec8e9f330c74429
+# TODO: unclear why this is here!
+Naming/FileName:
+  Description: 'Use snake_case for source file names.'
+  Enabled: false
+
+# Introduced in: c69a7eb3af955d6c4c0cf0c3cec8e9f330c74429
+# TODO: unclear why this is here!
+Naming/PredicateName:
+  Description: 'Check the names of predicate methods.'
+  Enabled: false

data/config/rails.yml

@@ -13,34 +13,32 @@ AllCops:
 Rails:
   Enabled: true
 
+# Introduced in: c69a7eb3af955d6c4c0cf0c3cec8e9f330c74429
+# TODO: unclear why this is here!
 Rails/ActionFilter:
   Description: 'Enforces consistent use of action filter methods.'
   Enabled: false
 
-Rails/Delegate:
-  Description: 'Prefer delegate method for delegations.'
-  Enabled: false
-
+# Introduced in: c69a7eb3af955d6c4c0cf0c3cec8e9f330c74429
+# TODO: unclear why this is here!
 Rails/HasAndBelongsToMany:
   Description: 'Prefer has_many :through to has_and_belongs_to_many.'
   Enabled: false
 
+# Introduced in: c69a7eb3af955d6c4c0cf0c3cec8e9f330c74429
+# TODO: unclear why this is here!
 Rails/Output:
   Description: 'Checks for calls to puts, print, etc.'
   Enabled: false
 
-Rails/ReadWriteAttribute:
-  Description: 'Checks for read_attribute(:attr) and write_attribute(:attr, val).'
-  Enabled: false
-
+# Introduced in: c69a7eb3af955d6c4c0cf0c3cec8e9f330c74429
+# TODO: unclear why this is here!
 Rails/ScopeArgs:
   Description: 'Checks the arguments of ActiveRecord scopes.'
   Enabled: false
 
-Rails/Validation:
-  Description: 'Use sexy validations.'
-  Enabled: false
-
+# Introduced in: 91d7bf4895db12727582ad7bf47bdcb20ab178f7
+# TODO: unclear (in any real detail) why this is here!
 Rails/SkipsModelValidations:
   Description: 'Avoid methods that skip model validations.'
   Enabled: false

data/config/rake.yml

@@ -0,0 +1 @@
+require: rubocop-rake

data/config/style.yml

@@ -0,0 +1,284 @@
+# https://github.com/alphagov/govuk-lint/pull/36
+# "When we have sequence of if/unless statements,
+# some with multiple lines within the if statement
+# block and some with a single line, forcing the single
+# line statements to re-written makes it a harder
+# to follow the branching logic."
+Style/IfUnlessModifier:
+  Description: Favor modifier if/unless usage when you have a single-line body.
+  Enabled: false
+
+# Part of the orignal GDS styleguide
+# "Never chain do...end"
+# https://github.com/alphagov/styleguides/blob/6395a10d41c3938f4c147cda443fd83f854c3e7a/ruby.md#syntax
+Style/MethodCalledOnDoEndBlock:
+  Description: Avoid chaining a method call on a do...end block.
+  Enabled: true
+
+# Part of the orignal GDS styleguide
+# "Omit the parentheses when the method doesn’t accept any arguments"
+# https://github.com/alphagov/styleguides/blob/6395a10d41c3938f4c147cda443fd83f854c3e7a/ruby.md#syntax
+Style/MethodCallWithoutArgsParentheses:
+  Description: 'Do not use parentheses for method calls with no arguments.'
+  Enabled: true
+
+# Part of the orignal GDS styleguide
+# "Use Ruby 1.9 syntax for symbolic hash keys.
+# This includes method calls."
+# https://github.com/alphagov/styleguides/blob/6395a10d41c3938f4c147cda443fd83f854c3e7a/ruby.md#collections
+Style/HashSyntax:
+  Exclude:
+  - 'db/schema.rb'
+
+# Part of the orignal GDS styleguide
+# "Add a trailing comma to multi-line array [...]
+# for clearer diffs with less line noise."
+# https://github.com/alphagov/styleguides/blob/6395a10d41c3938f4c147cda443fd83f854c3e7a/ruby.md#collections
+Style/TrailingCommaInArrayLiteral:
+  EnforcedStyleForMultiline: comma
+
+# Part of the orignal GDS styleguide
+# "Add a trailing comma to multi-line hash definitions [...]
+# for clearer diffs with less line noise."
+# https://github.com/alphagov/styleguides/blob/6395a10d41c3938f4c147cda443fd83f854c3e7a/ruby.md#collections
+Style/TrailingCommaInHashLiteral:
+  EnforcedStyleForMultiline: comma
+
+# Part of the orignal GDS styleguide
+# "When long lists of method arguments require
+# breaking over multiple lines, break each successive
+# argument on a new line, including the first argument
+# and closing paren. The final argument should include
+# a trailing comma."
+# https://github.com/alphagov/styleguides/blob/6395a10d41c3938f4c147cda443fd83f854c3e7a/ruby.md#general
+Style/TrailingCommaInArguments:
+  EnforcedStyleForMultiline: comma
+
+# Part of the orignal GDS styleguide
+# "Prefer %w to the literal array syntax when you need
+# an array of strings."
+# https://github.com/alphagov/styleguides/blob/6395a10d41c3938f4c147cda443fd83f854c3e7a/ruby.md#collections
+Style/WordArray:
+  MinSize: 0
+
+# "Try not to mix up single-quoted and double-quoted
+# strings within a file: it can make the code harder to read.
+# Definitely don't mix up single-quoted and double-quoted
+# strings within a method. If in doubt, use double-quoted strings,
+# because you’ll probably need to use interpolation somewhere.
+Style/StringLiterals:
+  EnforcedStyle: double_quotes
+
+# Introduced in: 5ca6b7d20fd62a6ce890868abdeca12837e436d7
+# "The wording of the description is hard to understand - it's not
+# immediately obvious what you have to do, and doesn't really say why
+# this is a good thing.
+#
+# It's not auto-fixable, which means it takes a lot of time to get the
+# syntax right for every occurrence of `%.2f` for example (taken from
+# `smart-answers`) for not very much benefit."
+Style/FormatStringToken:
+  Enabled: false
+
+# Introduced in: c69a7eb3af955d6c4c0cf0c3cec8e9f330c74429
+# TODO: unclear why this is here!
+Style/Alias:
+  Description: 'Use alias_method instead of alias.'
+  Enabled: false
+
+# Introduced in: c69a7eb3af955d6c4c0cf0c3cec8e9f330c74429
+# TODO: unclear why this is here!
+Style/AsciiComments:
+  Description: 'Use only ascii symbols in comments.'
+  Enabled: false
+
+# Introduced in: c69a7eb3af955d6c4c0cf0c3cec8e9f330c74429
+# TODO: unclear why this is here!
+Style/BeginBlock:
+  Description: 'Avoid the use of BEGIN blocks.'
+  Enabled: false
+
+# Introduced in: c69a7eb3af955d6c4c0cf0c3cec8e9f330c74429
+# TODO: unclear why this is here!
+Style/BlockDelimiters:
+  Description: >-
+    Avoid using {...} for multi-line blocks (multiline chaining is
+    always ugly).
+    Prefer {...} over do...end for single-line blocks.
+  Enabled: false
+
+# Introduced in: c69a7eb3af955d6c4c0cf0c3cec8e9f330c74429
+# TODO: unclear why this is here!
+Style/CaseEquality:
+  Description: 'Avoid explicit use of the case equality operator(===).'
+  Enabled: false
+
+# Introduced in: c69a7eb3af955d6c4c0cf0c3cec8e9f330c74429
+# TODO: unclear why this is here!
+Style/ClassAndModuleChildren:
+  Description: 'Checks style of children classes and modules.'
+  Enabled: false
+
+# Introduced in: c69a7eb3af955d6c4c0cf0c3cec8e9f330c74429
+# TODO: unclear why this is here!
+Style/PreferredHashMethods:
+  Description: 'Checks for use of deprecated Hash methods.'
+  Enabled: false
+
+# Introduced in: c69a7eb3af955d6c4c0cf0c3cec8e9f330c74429
+# Documenting every class is a lot of effort and we don't
+# expect to get any value from this. Another risk of adding
+# more documentation is the potential for confusion if that
+# documentation gets out-of-sync with the class/module.
+Style/Documentation:
+  Description: 'Document classes and non-namespace modules.'
+  Enabled: false
+
+# Introduced in: c69a7eb3af955d6c4c0cf0c3cec8e9f330c74429
+# TODO: unclear why this is here!
+Style/DoubleNegation:
+  Description: 'Checks for uses of double negation (!!).'
+  Enabled: false
+
+# Introduced in: c69a7eb3af955d6c4c0cf0c3cec8e9f330c74429
+# TODO: unclear why this is here!
+Style/Encoding:
+  Description: 'Use UTF-8 as the source file encoding.'
+  Enabled: false
+
+# TODO: duplicate (other-excludes.yml), remove
+Style/FormatString:
+  Description: 'Enforce the use of Kernel#sprintf, Kernel#format or String#%.'
+  Enabled: false
+
+# Introduced in: c69a7eb3af955d6c4c0cf0c3cec8e9f330c74429
+# TODO: unclear why this is here!
+Style/GlobalVars:
+  Description: 'Do not introduce global variables.'
+  Enabled: false
+
+# Introduced in: c69a7eb3af955d6c4c0cf0c3cec8e9f330c74429
+# This can lead to excessively long lines. Consistent with
+# disabling "Layout/LineLength".
+#
+# "There are occasions where following this rule forces you to make the
+# code less readable."
+Style/GuardClause:
+  Description: 'Check for conditionals that can be replaced with guard clauses'
+  Enabled: false
+
+# Introduced in: c69a7eb3af955d6c4c0cf0c3cec8e9f330c74429
+# TODO: unclear why this is here!
+Style/IfWithSemicolon:
+  Description: 'Never use if x; .... Use the ternary operator instead.'
+  Enabled: false
+
+# Introduced in: c69a7eb3af955d6c4c0cf0c3cec8e9f330c74429
+# TODO: unclear why this is here!
+Style/Lambda:
+  Description: 'Use the new lambda literal syntax for single-line blocks.'
+  Enabled: false
+
+# Introduced in: c69a7eb3af955d6c4c0cf0c3cec8e9f330c74429
+# TODO: unclear why this is here!
+Style/LambdaCall:
+  Description: 'Use lambda.call(...) instead of lambda.(...).'
+  Enabled: false
+
+# Introduced in: c69a7eb3af955d6c4c0cf0c3cec8e9f330c74429
+# TODO: unclear why this is here!
+Style/MethodDefParentheses:
+  Description: >-
+    Checks if the method definitions have or don't have
+    parentheses.
+  Enabled: false
+
+# Introduced in: c69a7eb3af955d6c4c0cf0c3cec8e9f330c74429
+# TODO: unclear why this is here!
+Style/ModuleFunction:
+  Description: 'Checks for usage of `extend self` in modules.'
+  Enabled: false
+
+# Introduced in: c69a7eb3af955d6c4c0cf0c3cec8e9f330c74429
+# TODO: unclear why this is here!
+Style/NegatedIf:
+  Description: >-
+    Favor unless over if for negative conditions
+    (or control flow or).
+  Enabled: false
+
+# Introduced in: c69a7eb3af955d6c4c0cf0c3cec8e9f330c74429
+# TODO: unclear why this is here!
+Style/Next:
+  Description: 'Use `next` to skip iteration instead of a condition at the end.'
+  Enabled: false
+
+# Introduced in: c69a7eb3af955d6c4c0cf0c3cec8e9f330c74429
+# TODO: unclear why this is here!
+Style/NilComparison:
+  Description: 'Prefer x.nil? to x == nil.'
+  Enabled: false
+
+# Introduced in: c69a7eb3af955d6c4c0cf0c3cec8e9f330c74429
+# TODO: unclear why this is here!
+Style/NumericLiterals:
+  Description: >-
+    Add underscores to large numeric literals to improve their
+    readability.
+  Enabled: false
+
+# Introduced in: c69a7eb3af955d6c4c0cf0c3cec8e9f330c74429
+# TODO: unclear why this is here! Suggest enabling with
+# "EnforcedStyle: compact"
+Style/RaiseArgs:
+  Description: 'Checks the arguments passed to raise/fail.'
+  Enabled: false
+
+# Analog of: 736b3d295f88b9ba6676fc168b823535582388c2
+# "Disable opinionated cops"
+#
+# We should avoid cops that are based on heuristics, since
+# it's not clear what action to take to fix an issue.
+Style/RegexpLiteral:
+  Description: >-
+    Use %r for regular expressions matching more than
+    `MaxSlashes` '/' characters.
+    Use %r only for regular expressions matching more than
+    `MaxSlashes` '/' character.
+  Enabled: false
+
+# Introduced in: 7aaebf4dbdf2a8d677b4000d3cd3512d4fb91e99
+# "This is a relatively new Ruby feature and is not mentioned in the Style
+# guide, so this commit disables it by default."
+# TODO: unclear why this is here!
+Style/SafeNavigation:
+  Description: >-
+    This cop transforms usages of a method call safeguarded by a check for the
+    existance of the object to safe navigation (`&.`).
+  Enabled: false
+
+# Introduced in: c69a7eb3af955d6c4c0cf0c3cec8e9f330c74429
+# TODO: unclear why this is here!
+Style/SignalException:
+  Description: 'Checks for proper usage of fail and raise.'
+  Enabled: false
+
+# Introduced in: c69a7eb3af955d6c4c0cf0c3cec8e9f330c74429
+# TODO: unclear why this is here! Suggest enabling with
+# "AllowIfMethodIsEmpty: true" to accommodate Rails actions.
+Style/SingleLineMethods:
+  Description: 'Avoid single-line methods.'
+  Enabled: false
+
+# Introduced in: c69a7eb3af955d6c4c0cf0c3cec8e9f330c74429
+# TODO: unclear why this is here!
+Style/CommandLiteral:
+  Description: 'Checks for %x when `` would do.'
+  Enabled: false
+
+# Introduced in: b171d652d3e434b74ddc621df3b5be24c49bc7e8
+# This cop was added in preperation for a Ruby feature
+# that is no longer likely to become part of the language.
+# https://github.com/rubocop-hq/rubocop/issues/7197
+Style/FrozenStringLiteralComment:
+  Enabled: false