rubocop-github 0.19.0 → 0.20.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 7e2de9cd12a2e6dc23c4ab4af442fa0989b99c0b2f775300ea9137aa1d569591
-  data.tar.gz: eede5dc50f6f7fdf7fd4d05cede6b70a989e97c2c2df18420b97fefd8997782e
+  metadata.gz: d23c4950774e5fe3dd1dea8eb07c44ab795b16987b3915eb50456cb9d4826ad3
+  data.tar.gz: 2003404eb801cc43c6ef0a7625f4536e334503577dafaa26d0ecaa3a0ea873e9
 SHA512:
-  metadata.gz: a8fc315cb6296196a0c3358990e2d5338217afcfbad7fc78a3dd636a52c32d3da2e2566906788fc8cf9818d3543d9939a59e431c3d7dc48eb105d58a8cb63f1e
-  data.tar.gz: da8ea70e7537cf46b80ccd81804bdbc407c45822a727c0715e571dc19f2706416ddefd6d1504da0f1a67254c4ef99bfdecc5054732eabbf1571a07a39d9c87e9
+  metadata.gz: 105a5f9824d72e61041aae7559753e008e4337cf3a1d2a0b646e649647bbf42d8ded35d1a495a757500c02de31e9d44aeef4e25476c5a0b10b045c5712ef893d
+  data.tar.gz: bd5d04dd886cf8804834fea5128036dc601d02c4f984c50e3f485ccf2bc8da329f27eda1a0ad0c088556fc4857c88d71f8a3a020b67176af99a562ed2d4eb5ca

data/README.md

@@ -1,6 +1,6 @@
 # RuboCop GitHub ![CI](https://github.com/github/rubocop-github/workflows/CI/badge.svg?event=push)
 
-This repository provides recommended RuboCop configuration and additional Cops for use on GitHub open source and internal Ruby projects.
+This repository provides recommended RuboCop configuration and additional Cops for use on GitHub open source and internal Ruby projects, and is the home of [GitHub's Ruby Style Guide](./STYLEGUIDE.md).
 
 ## Usage
 

data/STYLEGUIDE.md

@@ -27,6 +27,9 @@ This is GitHub's Ruby Style Guide, inspired by [RuboCop's guide][rubocop-guide].
     1. [Conditional keywords](#conditional-keywords)
     2. [Ternary operator](#ternary-operator)
 17. [Syntax](#syntax)
+18. [Rails](#rails)
+    1. [content_for](#content_for)
+    2. [Instance Variables in Views](#instance-variables-in-views)
 
 ## Layout
 
@@ -34,9 +37,11 @@ This is GitHub's Ruby Style Guide, inspired by [RuboCop's guide][rubocop-guide].
 
 * Use soft-tabs with a two space indent.
   <a name="default-indentation"></a><sup>[[link](#default-indentation)]</sup>
+  * <a href="https://docs.rubocop.org/rubocop/cops_layout.html#layoutindentationstyle">RuboCop rule: Layout/IndentationStyle</a>
 
 * Indent `when` with the start of the `case` expression.
   <a name="indent-when-as-start-of-case"></a><sup>[[link](#indent-when-as-start-of-case)]</sup>
+  * <a href="https://docs.rubocop.org/rubocop/cops_layout.html#layoutcaseindentation">RuboCop rule: Layout/CaseIndentation</a>
 
 ``` ruby
 # bad
@@ -80,10 +85,18 @@ end
 
 * Never leave trailing whitespace.
   <a name="trailing-whitespace"></a><sup>[[link](#trailing-whitespace)]</sup>
+  * <a href="https://docs.rubocop.org/rubocop/cops_layout.html#layouttrailingwhitespace">RuboCop rule: Layout/TrailingWhitespace</a>
 
 * Use spaces around operators, after commas, colons and semicolons, around `{`
   and before `}`.
   <a name="spaces-operators"></a><sup>[[link](#spaces-operators)]</sup>
+  * <a href="https://docs.rubocop.org/rubocop/cops_layout.html#layoutspacearoundoperators">RuboCop rule: Layout/SpaceAroundOperators</a>
+  * <a href="https://docs.rubocop.org/rubocop/cops_layout.html#layoutspaceaftercomma">RuboCop rule: Layout/SpaceAfterComma</a>
+  * <a href="https://docs.rubocop.org/rubocop/cops_layout.html#layoutspaceaftercolon">RuboCop rule: Layout/SpaceAfterColon</a>
+  * <a href="https://docs.rubocop.org/rubocop/cops_layout.html#layoutspacebeforeblockbraces">RuboCop rule: Layout/SpaceBeforeBlockBraces</a>
+  * <a href="https://docs.rubocop.org/rubocop/cops_layout.html#layoutspaceinsidehashliteralbraces">RuboCop rule: Layout/SpaceInsideHashLiteralBraces</a>
+  * <a href="https://docs.rubocop.org/rubocop/cops_style.html#stylehashsyntax">RuboCop rule: Style/HashSyntax</a>
+  * <a href="https://docs.rubocop.org/rubocop/cops_layout.html#layoutspacearoundoperators">RuboCop rule: Layout/SpaceAroundOperators</a>
 
 ``` ruby
 sum = 1 + 2
@@ -94,6 +107,8 @@ a, b = 1, 2
 
 * No spaces after `(`, `[` or before `]`, `)`.
   <a name="no-spaces-braces"></a><sup>[[link](#no-spaces-braces)]</sup>
+  * <a href="https://docs.rubocop.org/rubocop/cops_layout.html#layoutspaceinsideparens">RuboCop rule: Layout/SpaceInsideParens</a>
+  * <a href="https://docs.rubocop.org/rubocop/cops_layout.html#layoutspaceinsidereferencebrackets">RuboCop rule: Layout/SpaceInsideReferenceBrackets</a>
 
 ``` ruby
 some(arg).other
@@ -102,6 +117,7 @@ some(arg).other
 
 * No spaces after `!`.
   <a name="no-spaces-bang"></a><sup>[[link](#no-spaces-bang)]</sup>
+  * <a href="https://docs.rubocop.org/rubocop/cops_layout.html#layoutspaceafternot">RuboCop rule: Layout/SpaceAfterNot</a>
 
 ``` ruby
 !array.include?(element)
@@ -111,10 +127,12 @@ some(arg).other
 
 * End each file with a [newline](https://github.com/bbatsov/ruby-style-guide#newline-eof).
   <a name="newline-eof"></a><sup>[[link](#newline-eof)]</sup>
+  * <a href="https://docs.rubocop.org/rubocop/cops_layout.html#layouttrailingemptylines">RuboCop rule: Layout/TrailingEmptyLines</a>
 
 * Use empty lines between `def`s and to break up a method into logical
   paragraphs.
   <a name="empty-lines-def"></a><sup>[[link](#empty-lines-def)]</sup>
+  * <a href="https://docs.rubocop.org/rubocop/cops_layout.html#layoutemptylinebetweendefs">RuboCop rule: Layout/EmptyLineBetweenDefs</a>
 
 ``` ruby
 def some_method
@@ -134,12 +152,14 @@ end
 
 * Keep each line of code to a readable length. Unless you have a reason to, keep lines to a maximum of 118 characters. Why 118? That's the width at which the pull request diff UI needs horizontal scrolling (making pull requests harder to review).
   <a name="line-length"></a><sup>[[link](#line-length)]</sup>
+  * <a href="https://docs.rubocop.org/rubocop/cops_layout.html#layoutlinelength">RuboCop rule: Layout/LineLength</a>
 
 ## Classes
 
 * Avoid the usage of class (`@@`) variables due to their unusual behavior
 in inheritance.
   <a name="class-variables"></a><sup>[[link](#class-variables)]</sup>
+  * <a href="https://docs.rubocop.org/rubocop/cops_style.html#styleclassvars">RuboCop rule: Style/ClassVars</a>
 
 ``` ruby
 class Parent
@@ -164,6 +184,7 @@ Parent.print_class_var # => will print "child"
 * Use `def self.method` to define singleton methods. This makes the methods
   more resistant to refactoring changes.
   <a name="singleton-methods"></a><sup>[[link](#singleton-methods)]</sup>
+  * <a href="https://docs.rubocop.org/rubocop/cops_style.html#styleclassmethodsdefinitions">RuboCop rule: Style/ClassMethodsDefinitions</a>
 
 ``` ruby
 class TestClass
@@ -181,6 +202,7 @@ class TestClass
 * Avoid `class << self` except when necessary, e.g. single accessors and aliased
   attributes.
   <a name="class-method-definitions"></a><sup>[[link](#class-method-definitions)]</sup>
+  * <a href="https://docs.rubocop.org/rubocop/cops_style.html#styleclassmethodsdefinitions">RuboCop rule: Style/ClassMethodsDefinitions</a>
 
 ``` ruby
 class TestClass
@@ -214,6 +236,8 @@ end
 * Indent the `public`, `protected`, and `private` methods as much the
   method definitions they apply to. Leave one blank line above them.
   <a name="access-modifier-identation"></a><sup>[[link](#access-modifier-identation)]</sup>
+  * <a href="https://docs.rubocop.org/rubocop/cops_layout.html#layoutaccessmodifierindentation">RuboCop rule: Layout/AccessModifierIndentation</a>
+  * <a href="https://docs.rubocop.org/rubocop/cops_layout.html#layoutemptylinesaroundaccessmodifier">RuboCop rule: Layout/EmptyLinesAroundAccessModifier</a>
 
 ``` ruby
 class SomeClass
@@ -231,6 +255,7 @@ end
 * Avoid explicit use of `self` as the recipient of internal class or instance
   messages unless to specify a method shadowed by a variable.
   <a name="self-messages"></a><sup>[[link](#self-messages)]</sup>
+  * <a href="https://docs.rubocop.org/rubocop/cops_style.html#styleredundantself">RuboCop rule: Style/RedundantSelf</a>
 
 ``` ruby
 class SomeClass
@@ -248,6 +273,7 @@ end
 * Prefer `%w` to the literal array syntax when you need an array of
 strings.
   <a name="percent-w"></a><sup>[[link](#percent-w)]</sup>
+  * <a href="https://docs.rubocop.org/rubocop/cops_style.html#stylewordarray">RuboCop rule: Style/WordArray</a>
 
 ``` ruby
 # bad
@@ -265,6 +291,7 @@ STATES = %w(draft open closed)
 
 * Use symbols instead of strings as hash keys.
   <a name="symbols-as-keys"></a><sup>[[link](#symbols-as-keys)]</sup>
+  * <a href="https://docs.rubocop.org/rubocop/cops_style.html#stylestringhashkeys">RuboCop rule: Style/StringHashKeys</a>
 
 ``` ruby
 # bad
@@ -300,9 +327,10 @@ end
 
 Avoid calling `send` and its cousins unless you really need it. Metaprogramming can be extremely powerful, but in most cases you can write code that captures your meaning by being explicit:
 <a name="avoid-send"></a><sup>[[link](#avoid-send)]</sup>
+  * <a href="https://docs.rubocop.org/rubocop/cops_style.html#stylesend">RuboCop rule: Style/Send</a>
 
 ``` ruby
-# avoid 
+# avoid
 unless [:base, :head].include?(base_or_head)
   raise ArgumentError, "base_or_head must be either :base or :head"
 end
@@ -366,6 +394,7 @@ end
 
 Use the Ruby 1.9 syntax for hash literals when all the keys are symbols:
 <a name="symbols-as-hash-keys"></a><sup>[[link](#symbols-as-hash-keys)]</sup>
+* <a href="https://docs.rubocop.org/rubocop/cops_style.html#stylestringhashkeys">RuboCop rule: Style/StringHashKeys</a>
 
 ``` ruby
 # bad
@@ -396,6 +425,7 @@ link_to("Account", controller: "users", action: "show", id: user)
 
 If you have a hash with mixed key types, use the legacy hashrocket style to avoid mixing styles within the same hash:
 <a name="consistent-hash-syntax"></a><sup>[[link](#consistent-hash-syntax)]</sup>
+* <a href="https://docs.rubocop.org/rubocop/cops_style.html#stylehashsyntax">RuboCop rule: Style/HashSyntax</a>
 
 ``` ruby
 
@@ -417,6 +447,7 @@ hsh = {
 
 [Keyword arguments](http://magazine.rubyist.net/?Ruby200SpecialEn-kwarg) are recommended but not required when a method's arguments may otherwise be opaque or non-obvious when called. Additionally, prefer them over the old "Hash as pseudo-named args" style from pre-2.0 ruby.
 <a name="keyword-arguments"></a><sup>[[link](#keyword-arguments)]</sup>
+* <a href="https://docs.rubocop.org/rubocop/cops_style.html#styleoptionalbooleanparameter">RuboCop rule: Style/OptionalBooleanParameter</a>
 
 So instead of this:
 
@@ -444,21 +475,26 @@ remove_member(user, skip_membership_check: true)
 
 * Use `snake_case` for methods and variables.
   <a name="snake-case-methods-vars"></a><sup>[[link](#snake-case-methods-vars)]</sup>
+  * <a href="https://docs.rubocop.org/rubocop/cops_naming.html#namingsnakecase">RuboCop rule: Naming/SnakeCase</a>
+  * <a href="https://docs.rubocop.org/rubocop/cops_naming.html#namingvariablename">RuboCop rule: Naming/VariableName</a>
 
 * Use `CamelCase` for classes and modules.  (Keep acronyms like HTTP,
   RFC, XML uppercase.)
   <a name="camelcase-classes-modules"></a><sup>[[link](#camelcase-classes-modules)]</sup>
+  * <a href="https://docs.rubocop.org/rubocop/cops_naming.html#namingclassandmodulecamelcase">RuboCop rule: Naming/ClassAndModuleCamelCase</a>
 
 * Use `SCREAMING_SNAKE_CASE` for other constants.
   <a name="screaming-snake-case-constants"></a><sup>[[link](#screaming-snake-case-constants)]</sup>
+  * <a href="https://docs.rubocop.org/rubocop/cops_naming.html#namingconstantname">RuboCop rule: Naming/ConstantName</a>
 
 * The names of predicate methods (methods that return a boolean value)
   should end in a question mark. (i.e. `Array#empty?`).
   <a name="bool-methods-qmark"></a><sup>[[link](#bool-methods-qmark)]</sup>
+  * <a href="https://docs.rubocop.org/rubocop/cops_naming.html#namingpredicatename">RuboCop rule: Naming/PredicateName</a>
 
 * The names of potentially "dangerous" methods (i.e. methods that modify `self` or the
   arguments, `exit!`, etc.) should end with an exclamation mark. Bang methods
-  should only exist if a non-bang counterpart (method name which does NOT end with !) 
+  should only exist if a non-bang counterpart (method name which does NOT end with !)
   also exists.
   <a name="dangerous-method-bang"></a><sup>[[link](#dangerous-method-bang)]</sup>
 
@@ -466,6 +502,7 @@ remove_member(user, skip_membership_check: true)
 
 * Use `%w` freely.
   <a name="use-percent-w-freely"></a><sup>[[link](#use-percent-w-freely)]</sup>
+  * <a href="https://docs.rubocop.org/rubocop/cops_style.html#stylewordarray">RuboCop rule: Style/WordArray</a>
 
 ``` ruby
 STATES = %w(draft open closed)
@@ -474,6 +511,7 @@ STATES = %w(draft open closed)
 * Use `%()` for single-line strings which require both interpolation
   and embedded double-quotes. For multi-line strings, prefer heredocs.
   <a name="percent-parens-single-line"></a><sup>[[link](#percent-parens-single-line)]</sup>
+  * <a href="https://docs.rubocop.org/rubocop/cops_style.html#stylebarepercentliterals">RuboCop rule: Style/BarePercentLiterals</a>
 
 ``` ruby
 # bad (no interpolation needed)
@@ -494,6 +532,7 @@ STATES = %w(draft open closed)
 
 * Use `%r` only for regular expressions matching *more than* one '/' character.
   <a name="percent-r-regular-expressions"></a><sup>[[link](#percent-r-regular-expressions)]</sup>
+  * <a href="https://docs.rubocop.org/rubocop/cops_style.html#styleregexpliteral">RuboCop rule: Style/RegexpLiteral</a>
 
 ``` ruby
 # bad
@@ -512,7 +551,7 @@ STATES = %w(draft open closed)
 * Avoid using $1-9 as it can be hard to track what they contain. Named groups
   can be used instead.
   <a name="capture-with-named-groups"></a><sup>[[link](#capture-with-named-groups)]</sup>
-
+  * <a href="https://docs.rubocop.org/rubocop/cops_lint.html#mixedregexpcapturetypes">RuboCop rule: Lint/MixedRegexpCaptureTypes</a>
 ``` ruby
 # bad
 /(regexp)/ =~ string
@@ -571,6 +610,7 @@ documentation about the libraries that the current file uses.
 
 * Prefer string interpolation instead of string concatenation:
   <a name="string-interpolation"></a><sup>[[link](#string-interpolation)]</sup>
+  * <a href="https://docs.rubocop.org/rubocop/cops_style.html#stylestringconcatenation">RuboCop rule: Style/StringConcatenation</a>
 
 ``` ruby
 # bad
@@ -584,6 +624,7 @@ email_with_name = "#{user.name} <#{user.email}>"
   will always work without a delimiter change, and `'` is a lot more
   common than `"` in string literals.
   <a name="double-quotes"></a><sup>[[link](#double-quotes)]</sup>
+  * <a href="https://docs.rubocop.org/rubocop/cops_style.html#stylestringliterals">RuboCop rule: Style/StringLiterals</a>
 
 ``` ruby
 # bad
@@ -615,6 +656,7 @@ end
 * Use `def` with parentheses when there are arguments. Omit the
   parentheses when the method doesn't accept any arguments.
   <a name="method-parens-when-arguments"></a><sup>[[link](#method-parens-when-arguments)]</sup>
+  * <a href="https://docs.rubocop.org/rubocop/cops_style.html#styledefwithparentheses">RuboCop rule: Style/DefWithParentheses</a>
 
  ``` ruby
  def some_method
@@ -632,9 +674,11 @@ end
   always use parentheses in the method invocation. For example, write
   `f((3 + 2) + 1)`.
   <a name="parens-no-spaces"></a><sup>[[link](#parens-no-spaces)]</sup>
+  * <a href="https://docs.rubocop.org/rubocop/cops_style.html#stylemethodcallwithargsparentheses">RuboCop rule: Style/MethodCallWithArgsParentheses</a>
 
 * Never put a space between a method name and the opening parenthesis.
   <a name="no-spaces-method-parens"></a><sup>[[link](#no-spaces-method-parens)]</sup>
+  * <a href="https://docs.rubocop.org/rubocop/cops_style.html#styleparenthesesasgroupedexpression">RuboCop rule: Style/ParenthesesAsGroupedExpression</a>
 
 ``` ruby
 # bad
@@ -650,6 +694,7 @@ f(3 + 2) + 1
 
 * Never use `then` for multi-line `if/unless`.
   <a name="no-then-for-multi-line-if-unless"></a><sup>[[link](#no-then-for-multi-line-if-unless)]</sup>
+  * <a href="https://docs.rubocop.org/rubocop/cops_style.html#stylemultilineifthen">RuboCop rule: Style/MultilineIfThen</a>
 
 ``` ruby
 # bad
@@ -665,10 +710,12 @@ end
 
 * The `and` and `or` keywords are banned. It's just not worth it. Always use `&&` and `||` instead.
   <a name="no-and-or-or"></a><sup>[[link](#no-and-or-or)]</sup>
+  * <a href="https://docs.rubocop.org/rubocop/cops_style.html#styleandor">RuboCop rule: Style/AndOr</a>
 
 * Favor modifier `if/unless` usage when you have a single-line
   body.
   <a name="favor-modifier-if-unless"></a><sup>[[link](#favor-modifier-if-unless)]</sup>
+  * <a href="https://docs.rubocop.org/rubocop/cops_style.html#stylemultilineternaryoperator">RuboCop rule: Style/MultilineTernaryOperator</a>
 
 ``` ruby
 # bad
@@ -682,6 +729,7 @@ do_something if some_condition
 
 * Never use `unless` with `else`. Rewrite these with the positive case first.
   <a name="no-else-with-unless"></a><sup>[[link](#no-else-with-unless)]</sup>
+  * <a href="https://docs.rubocop.org/rubocop/cops_style.html#styleunlesselse">RuboCop rule: Style/UnlessElse</a>
 
 ``` ruby
 # bad
@@ -701,6 +749,7 @@ end
 
 * Don't use parentheses around the condition of an `if/unless/while`.
   <a name="no-parens-if-unless-while"></a><sup>[[link](#no-parens-if-unless-while)]</sup>
+  * <a href="https://docs.rubocop.org/rubocop/cops_style.html#styleparenthesesaroundcondition">RuboCop rule: Style/ParenthesesAroundCondition</a>
 
 ``` ruby
 # bad
@@ -720,6 +769,7 @@ end
   trivial. However, do use the ternary operator(`?:`) over `if/then/else/end` constructs
   for single line conditionals.
   <a name="trivial-ternary"></a><sup>[[link](#trivial-ternary)]</sup>
+  * <a href="https://docs.rubocop.org/rubocop/cops_style.html#stylemultilineternaryoperator">RuboCop rule: Style/MultilineTernaryOperator</a>
 
 ``` ruby
 # bad
@@ -731,11 +781,13 @@ result = some_condition ? something : something_else
 
 * Avoid multi-line `?:` (the ternary operator), use `if/unless` instead.
   <a name="no-multiline-ternary"></a><sup>[[link](#no-multiline-ternary)]</sup>
+  * <a href="https://docs.rubocop.org/rubocop/cops_style.html#stylemultilineternaryoperator">RuboCop rule: Style/MultilineTernaryOperator</a>
 
 * Use one expression per branch in a ternary operator. This
   also means that ternary operators must not be nested. Prefer
   `if/else` constructs in these cases.
   <a name="one-expression-per-branch"></a><sup>[[link](#one-expression-per-branch)]</sup>
+  * <a href="https://docs.rubocop.org/rubocop/cops_style.html#stylenestedternaryoperator">RuboCop rule: Style/NestedTernaryOperator</a>
 
 ``` ruby
 # bad
@@ -757,6 +809,7 @@ end
   doesn't introduce a new scope (unlike `each`) and variables defined
   in its block will be visible outside it.
   <a name="avoid-for"></a><sup>[[link](#avoid-for)]</sup>
+  * <a href="https://docs.rubocop.org/rubocop/cops_style.html#stylefor">RuboCop rule: Style/For</a>
 
 ``` ruby
 arr = [1, 2, 3]
@@ -776,6 +829,7 @@ arr.each { |elem| puts elem }
   definitions" (e.g. in Rakefiles and certain DSLs).  Avoid `do...end`
   when chaining.
   <a name="squiggly-braces"></a><sup>[[link](#squiggly-braces)]</sup>
+  * <a href="https://docs.rubocop.org/rubocop/cops_style.html#styleblockdelimiters">RuboCop rule: Style/BlockDelimiters</a>
 
 ``` ruby
 names = ["Bozhidar", "Steve", "Sarah"]
@@ -798,11 +852,12 @@ end.map { |name| name.upcase }
 ```
 
 * Some will argue that multiline chaining would look OK with the use of `{...}`,
- but they should ask themselves: is this code really readable and can't the block's 
+ but they should ask themselves: is this code really readable and can't the block's
  contents be extracted into nifty methods?
 
 * Avoid `return` where not required.
   <a name="avoid-return"></a><sup>[[link](#avoid-return)]</sup>
+  * <a href="https://docs.rubocop.org/rubocop/cops_style.html#styleredundantreturn">RuboCop rule: Style/RedundantReturn</a>
 
 ``` ruby
 # bad
@@ -818,6 +873,7 @@ end
 
 * Use spaces around the `=` operator when assigning default values to method parameters:
   <a name="spaces-around-equals"></a><sup>[[link](#spaces-around-equals)]</sup>
+  * <a href="https://docs.rubocop.org/rubocop/cops_style.html#stylespacearoundequalsinparameterdefault">RuboCop rule: Style/SpaceAroundEqualsInParameterDefault</a>
 
 ``` ruby
 # bad
@@ -850,6 +906,7 @@ if (v = next_value) == "hello" ...
 
 * Use `||=` freely to initialize variables.
   <a name="memoization-for-initialization"></a><sup>[[link](#memoize-away)]</sup>
+  * <a href="https://docs.rubocop.org/rubocop/cops_style.html#styleorassignment">RuboCop rule: Style/OrAssignment</a>
 
 ``` ruby
 # set name to Bozhidar, only if it's nil or false
@@ -859,6 +916,7 @@ name ||= "Bozhidar"
 * Don't use `||=` to initialize boolean variables. (Consider what
   would happen if the current value happened to be `false`.)
   <a name="no-memoization-for-boolean"></a><sup>[[link](#no-memoization-for-boolean)]</sup>
+  * <a href="https://docs.rubocop.org/rubocop/cops_style.html#styleorassignment">RuboCop rule: Style/OrAssignment</a>
 
 ``` ruby
 # bad - would set enabled to true even if it was false
@@ -873,9 +931,11 @@ enabled = true if enabled.nil?
   one-liner scripts is discouraged. Prefer long form versions such as
   `$PROGRAM_NAME`.
   <a name="no-cryptic-vars"></a><sup>[[link](#no-cryptic-vars)]</sup>
+  * <a href="https://docs.rubocop.org/rubocop/cops_style.html#stylespecialglobalvars">RuboCop rule: Style/SpecialGlobalVars</a>
 
 * Use `_` for unused block parameters.
   <a name="underscore-unused-vars"></a><sup>[[link](#underscore-unused-vars)]</sup>
+  * <a href="https://docs.rubocop.org/rubocop/cops_lint.html#lintunusedblockargument">RuboCop rule: Lint/UnusedBlockArgument</a>
 
 ``` ruby
 # bad
@@ -890,7 +950,133 @@ result = hash.map { |_, v| v + 1 }
   For example, `String === "hi"` is true and `"hi" === String` is false.
   Instead, use `is_a?` or `kind_of?` if you must.
   <a name="type-checking-is-a-kind-of"></a><sup>[[link](#type-checking-is-a-kind-of)]</sup>
+  * <a href="https://docs.rubocop.org/rubocop/cops_style.html#stylecaseequality">RuboCop rule: Style/CaseEquality</a>
 
   Refactoring is even better. It's worth looking hard at any code that explicitly checks types.
 
+## Rails
+
+### content_for
+
+Limit usage of `content_for` helper. The use of `content_for` is the same as setting an instance variable plus `capture`.
+
+``` erb
+<% content_for :foo do %>
+  Hello
+<% end %>
+```
+
+Is effectively the same as
+
+``` erb
+<% @foo_content = capture do %>
+  Hello
+<% end %>
+```
+
+See "Instance Variables in Views" below.
+
+#### Common Anti-patterns
+
+**Using `content_for` within the same template to capture data.**
+
+Instead, just use `capture`.
+
+``` erb
+<!-- bad -->
+<% content_for :page do %>
+  Hello
+<% end %>
+<% if foo? %>
+  <div class="container">
+    <%= yield :page %>
+  </div>
+<% else %>
+  <%= yield :page %>
+<% end %>
+```
+
+Simply capture and use a local variable since the result is only needed in this template.
+
+``` erb
+<!-- good -->
+<% page = capture do %>
+  Hello
+<% end %>
+<% if foo? %>
+  <div class="container">
+    <%= page %>
+  </div>
+<% else %>
+  <%= page %>
+<% end %>
+```
+
+**Using `content_for` to pass content to a subtemplate.**
+
+Instead, `render layout:` with a block.
+
+``` erb
+<!-- bad -->
+<% content_for :page do %>
+  Hello
+<% end %>
+<%= render partial: "page" %>
+<!-- _page.html.erb -->
+<div class="container">
+  <%= yield :page %>
+</div>
+```
+
+Pass the content in a block directly to the `render` function.
+
+``` erb
+<!-- good -->
+<%= render layout: "page" do %>
+  Hello
+<% end %>
+<!-- _page.html.erb -->
+<div class="container">
+  <%= yield %>
+</div>
+```
+
+### Instance Variables in Views
+
+In general, passing data between templates with instance variables is discouraged. This even applies from controllers to templates, not just between partials.
+
+`:locals` can be used to pass data from a controller just like partials.
+
+``` ruby
+def show
+  render "blob/show", locals: {
+    :repository => current_repository,
+    :commit     => current_commit,
+    :blob       => current_blob
+  }
+end
+```
+
+Rails implicit renders are also discouraged.
+
+Always explicitly render templates with a full directory path. This makes template callers easier to trace. You can find all the callers of `"app/view/site/hompage.html.erb"` with a simple project search for `"site/homepage"`.
+
+``` ruby
+def homepage
+  render "site/homepage"
+end
+```
+
+#### Exceptions
+
+There are some known edge cases where you might be forced to use instance variables. In these cases, its okay to do so.
+
+##### Legacy templates
+
+If you need to call a subview that expects an instance variable be set. If possible consider refactoring the subview to accept a local instead.
+
+##### Layouts
+
+Unfortunately the only way to get data into a layout template is with instance variables. You can't explicitly pass locals to them.
+
 [rubocop-guide]: https://github.com/rubocop-hq/ruby-style-guide

data/config/default.yml

@@ -1,3 +1,6 @@
+inherit_from:
+  - ./default_pending.yml
+
 require:
   - rubocop-github
   - rubocop-performance
@@ -65,8 +68,10 @@ Layout/BlockAlignment:
 Layout/BlockEndNewline:
   Enabled: true
 
+# TODO: Enable this since it's in the styleguide.
 Layout/CaseIndentation:
   Enabled: false
+  StyleGuide: https://github.com/github/rubocop-github/blob/main/STYLEGUIDE.md#indent-when-as-start-of-case
 
 Layout/ClassStructure:
   Enabled: false
@@ -180,16 +185,16 @@ Layout/HeredocIndentation:
   Enabled: false
 
 Layout/IndentationConsistency:
-  Enabled: false
+  StyleGuide: https://github.com/github/rubocop-github/blob/main/STYLEGUIDE.md#indentation
 
 Layout/IndentationStyle:
-  Enabled: true
   EnforcedStyle: spaces
   IndentationWidth: 2
+  StyleGuide: https://github.com/github/rubocop-github/blob/main/STYLEGUIDE.md#default-indentation
 
 Layout/IndentationWidth:
-  Enabled: true
   Width: 2
+  StyleGuide: https://github.com/github/rubocop-github/blob/main/STYLEGUIDE.md#default-indentation
 
 Layout/InitialIndentation:
   Enabled: true
@@ -258,10 +263,10 @@ Layout/SingleLineBlockChain:
   Enabled: false
 
 Layout/SpaceAfterColon:
-  Enabled: true
+  StyleGuide: https://github.com/github/rubocop-github/blob/main/STYLEGUIDE.md#spaces-operators
 
 Layout/SpaceAfterComma:
-  Enabled: true
+  StyleGuide: https://github.com/github/rubocop-github/blob/main/STYLEGUIDE.md#spaces-operators
 
 Layout/SpaceAfterMethodName:
   Enabled: true
@@ -270,16 +275,16 @@ Layout/SpaceAfterNot:
   Enabled: true
 
 Layout/SpaceAfterSemicolon:
-  Enabled: true
+  StyleGuide: https://github.com/github/rubocop-github/blob/main/STYLEGUIDE.md#spaces-operators
 
 Layout/SpaceAroundBlockParameters:
   Enabled: true
 
 Layout/SpaceAroundEqualsInParameterDefault:
-  Enabled: true
+  StyleGuide: https://github.com/github/rubocop-github/blob/main/STYLEGUIDE.md#spaces-around-equals
 
 Layout/SpaceAroundKeyword:
-  Enabled: false
+  StyleGuide: https://github.com/github/rubocop-github/blob/main/STYLEGUIDE.md#spaces-operators
 
 Layout/SpaceAroundMethodCallOperator:
   Enabled: false
@@ -309,8 +314,8 @@ Layout/SpaceInLambdaLiteral:
   Enabled: false
 
 Layout/SpaceInsideArrayLiteralBrackets:
-  Enabled: true
   EnforcedStyle: no_space
+  StyleGuide: https://github.com/github/rubocop-github/blob/main/STYLEGUIDE.md#no-spaces-braces
 
 Layout/SpaceInsideArrayPercentLiteral:
   Enabled: true
@@ -340,7 +345,7 @@ Layout/TrailingEmptyLines:
   Enabled: true
 
 Layout/TrailingWhitespace:
-  Enabled: true
+  StyleGuide: https://github.com/github/rubocop-github/blob/main/STYLEGUIDE.md#trailing-whitespace
 
 Lint/AmbiguousAssignment:
   Enabled: false
@@ -444,6 +449,15 @@ Lint/EmptyConditionalBody:
 Lint/EmptyEnsure:
   Enabled: true
 
+Lint/EmptyExpression:
+  Enabled: false
+
+Lint/EmptyFile:
+  Enabled: false
+
+Lint/EmptyWhen:
+  Enabled: false
+
 Lint/EmptyInterpolation:
   Enabled: true
 
@@ -597,6 +611,9 @@ Lint/RegexpAsCondition:
 Lint/RequireParentheses:
   Enabled: true
 
+Lint/RequireRangeParentheses:
+  Enabled: false
+
 Lint/RequireRelativeSelfPath:
   Enabled: false
 
@@ -678,8 +695,10 @@ Lint/UnreachableCode:
 Lint/UnreachableLoop:
   Enabled: false
 
+# TODO: Enable this since it's in the styleguide.
 Lint/UnusedBlockArgument:
   Enabled: false
+  StyleGuide: https://github.com/github/rubocop-github/blob/main/STYLEGUIDE.md#underscore-unused-vars
 
 Lint/UnusedMethodArgument:
   Enabled: false
@@ -976,8 +995,10 @@ Style/AccessorGrouping:
 Style/Alias:
   Enabled: false
 
+# TODO: Enable this since it's in the styleguide.
 Style/AndOr:
   Enabled: false
+  StyleGuide: https://github.com/github/rubocop-github/blob/main/STYLEGUIDE.md#no-and-or-or
 
 Style/ArgumentsForwarding:
   Enabled: false
@@ -1076,7 +1097,7 @@ Style/DateTime:
   Enabled: false
 
 Style/DefWithParentheses:
-  Enabled: true
+  StyleGuide: https://github.com/github/rubocop-github/blob/main/STYLEGUIDE.md#method-parens-when-arguments
 
 Style/Dir:
   Enabled: false
@@ -1114,6 +1135,9 @@ Style/EmptyCaseCondition:
 Style/EmptyElse:
   Enabled: false
 
+Style/EmptyHeredoc:
+  Enabled: false
+
 Style/EmptyLambdaParameter:
   Enabled: false
 
@@ -1163,7 +1187,7 @@ Style/FloatDivision:
   Enabled: false
 
 Style/For:
-  Enabled: true
+  StyleGuide: https://github.com/github/rubocop-github/blob/main/STYLEGUIDE.md#avoid-for
 
 Style/FormatString:
   Enabled: false
@@ -1199,8 +1223,8 @@ Style/HashLikeCase:
   Enabled: false
 
 Style/HashSyntax:
-  Enabled: true
   EnforcedStyle: ruby19_no_mixed_keys
+  StyleGuide: https://github.com/github/rubocop-github/blob/main/STYLEGUIDE.md#symbols-as-keys
 
 Style/HashTransformKeys:
   Enabled: false
@@ -1256,6 +1280,9 @@ Style/LambdaCall:
 Style/LineEndConcatenation:
   Enabled: false
 
+Style/MagicCommentFormat:
+  Enabled: false
+
 Style/MapCompactWithConditionalBlock:
   Enabled: false
 
@@ -1299,7 +1326,7 @@ Style/MultilineIfModifier:
   Enabled: false
 
 Style/MultilineIfThen:
-  Enabled: true
+  StyleGuide: https://github.com/github/rubocop-github/blob/main/STYLEGUIDE.md#no-then-for-multi-line-if-unless
 
 Style/MultilineInPatternThen:
   Enabled: false
@@ -1475,8 +1502,10 @@ Style/RedundantRegexpCharacterClass:
 Style/RedundantRegexpEscape:
   Enabled: false
 
+# TODO: Enable this since it's in the styleguide.
 Style/RedundantReturn:
   Enabled: false
+  StyleGuide: https://github.com/github/rubocop-github/blob/main/STYLEGUIDE.md#avoid-return
 
 Style/RedundantSelf:
   Enabled: false
@@ -1563,8 +1592,8 @@ Style/StringHashKeys:
   Enabled: false
 
 Style/StringLiterals:
-  Enabled: true
   EnforcedStyle: double_quotes
+  StyleGuide: https://github.com/github/rubocop-github/blob/main/STYLEGUIDE.md#double-quotes
 
 Style/StringLiteralsInInterpolation:
   Enabled: false
@@ -1626,8 +1655,10 @@ Style/TrailingUnderscoreVariable:
 Style/TrivialAccessors:
   Enabled: false
 
+# TODO: Enable this since it's in the styleguide.
 Style/UnlessElse:
   Enabled: false
+  StyleGuide: https://github.com/github/rubocop-github/blob/main/STYLEGUIDE.md#no-else-with-unless
 
 Style/UnlessLogicalOperators:
   Enabled: false

data/config/default_pending.yml

@@ -0,0 +1,6 @@
+Lint/DuplicateMagicComment: # new in 1.37
+  Enabled: false
+Style/OperatorMethodCall: # new in 1.37
+  Enabled: false
+Style/RedundantStringEscape: # new in 1.37
+  Enabled: false

data/config/rails.yml

@@ -1,10 +1,10 @@
+inherit_from:
+  - ./rails_pending.yml
+
 require:
   - rubocop-github-rails
   - rubocop-rails
 
-GitHub/RailsApplicationRecord:
-  Enabled: true
-
 GitHub/RailsControllerRenderActionSymbol:
   Enabled: true
 
@@ -17,9 +17,6 @@ GitHub/RailsControllerRenderPathsExist:
 GitHub/RailsControllerRenderShorthand:
   Enabled: true
 
-GitHub/RailsRenderInline:
-  Enabled: true
-
 GitHub/RailsRenderObjectCollection:
   Enabled: false
 
@@ -34,27 +31,27 @@ GitHub/RailsViewRenderShorthand:
 
 Layout/BlockAlignment:
   Exclude:
-    - app/views/**/*.erb
+    - "**/*.erb"
 
 Layout/IndentationWidth:
   Exclude:
-    - app/views/**/*.erb
+    - "**/*.erb"
 
 Layout/InitialIndentation:
   Exclude:
-    - app/views/**/*.erb
+    - "**/*.erb"
 
 Layout/SpaceInsideParens:
   Exclude:
-    - app/views/**/*.erb
+    - "**/*.erb"
 
 Layout/TrailingEmptyLines:
   Exclude:
-    - app/views/**/*.erb
+    - "**/*.erb"
 
 Layout/TrailingWhitespace:
   Exclude:
-    - app/views/**/*.erb
+    - "**/*.erb"
 
 Lint/UselessAccessModifier:
   ContextCreatingMethods:
@@ -94,7 +91,7 @@ Rails/ApplicationMailer:
   Enabled: false
 
 Rails/ApplicationRecord:
-  Enabled: false
+  Enabled: true
 
 Rails/ArelStar:
   Enabled: false
@@ -307,7 +304,7 @@ Rails/RelativeDateConstant:
   Enabled: false
 
 Rails/RenderInline:
-  Enabled: false
+  Enabled: true
 
 Rails/RenderPlainText:
   Enabled: false
@@ -399,16 +396,16 @@ Rails/WhereNot:
 
 Style/For:
   Exclude:
-    - app/views/**/*.erb
+    - "**/*.erb"
 
 Style/OneLineConditional:
   Exclude:
-    - app/views/**/*.erb
+    - "**/*.erb"
 
 Style/Semicolon:
   Exclude:
-    - app/views/**/*.erb
+    - "**/*.erb"
 
 Style/StringLiterals:
   Exclude:
-    - app/views/**/*.erb
+    - "**/*.erb"

data/config/rails_cops.yml

@@ -1,6 +1,3 @@
-GitHub/RailsApplicationRecord:
-  Enabled: pending
-
 GitHub/RailsControllerRenderActionSymbol:
   Enabled: pending
   Include:
@@ -8,7 +5,7 @@ GitHub/RailsControllerRenderActionSymbol:
 
 GitHub/RailsControllerRenderLiteral:
   Enabled: pending
-  StyleGuide: https://github.com/github/rubocop-github/blob/master/guides/rails-render-literal.md
+  StyleGuide: https://github.com/github/rubocop-github/blob/main/guides/rails-render-literal.md
   Include:
     - 'app/controllers/**/*.rb'
 
@@ -21,25 +18,16 @@ GitHub/RailsControllerRenderPathsExist:
 
 GitHub/RailsControllerRenderShorthand:
   Enabled: pending
-  StyleGuide: https://github.com/github/rubocop-github/blob/master/guides/rails-controller-render-shorthand.md
+  StyleGuide: https://github.com/github/rubocop-github/blob/main/guides/rails-controller-render-shorthand.md
   Include:
     - 'app/controllers/**/*.rb'
 
-GitHub/RailsRenderInline:
-  Enabled: pending
-  StyleGuide: https://github.com/github/rubocop-github/blob/master/guides/rails-controller-render-inline.md
-  Include:
-    - 'app/controllers/**/*.rb'
-    - 'app/helpers/**/*.rb'
-    - 'app/view_models/**/*.rb'
-    - 'app/views/**/*.erb'
-
 GitHub/RailsRenderObjectCollection:
   Enabled: pending
 
 GitHub/RailsViewRenderLiteral:
   Enabled: pending
-  StyleGuide: https://github.com/github/rubocop-github/blob/master/guides/rails-render-literal.md
+  StyleGuide: https://github.com/github/rubocop-github/blob/main/guides/rails-render-literal.md
   Include:
     - 'app/helpers/**/*.rb'
     - 'app/view_models/**/*.rb'

data/config/rails_pending.yml

@@ -0,0 +1,20 @@
+Rails/ActionControllerFlashBeforeRender: # new in 2.16
+  Enabled: false
+Rails/ActionOrder: # new in 2.17
+  Enabled: false
+Rails/ActiveSupportOnLoad: # new in 2.16
+  Enabled: false
+Rails/FreezeTime: # new in 2.16
+  Enabled: false
+Rails/IgnoredColumnsAssignment: # new in 2.17
+  Enabled: false
+Rails/RootPathnameMethods: # new in 2.16
+  Enabled: false
+Rails/ToSWithArgument: # new in 2.16
+  Enabled: false
+Rails/TopLevelHashWithIndifferentAccess: # new in 2.16
+  Enabled: false
+Rails/WhereMissing: # new in 2.16
+  Enabled: false
+Rails/WhereNotWithMultipleConditions: # new in 2.17
+  Enabled: false

data/lib/rubocop/cop/github/rails_view_render_shorthand.rb

@@ -6,6 +6,8 @@ module RuboCop
   module Cop
     module GitHub
       class RailsViewRenderShorthand < Base
+        extend AutoCorrector
+
         MSG = "Prefer `render` partial shorthand"
 
         def_node_matcher :render_with_options?, <<-PATTERN
@@ -26,9 +28,13 @@ module RuboCop
             locals_key = option_pairs.map { |pair| locals_key?(pair) }.compact.first
 
             if option_pairs.length == 1 && partial_key
-              add_offense(node, message: "Use `render #{partial_key.source}` instead")
+              add_offense(node, message: "Use `render #{partial_key.source}` instead") do |corrector|
+                corrector.replace(node.source_range, "render #{partial_key.source}")
+              end
             elsif option_pairs.length == 2 && partial_key && locals_key
-              add_offense(node, message: "Use `render #{partial_key.source}, #{locals_key.source}` instead")
+              add_offense(node, message: "Use `render #{partial_key.source}, #{locals_key.source}` instead") do |corrector|
+                corrector.replace(node.source_range, "render #{partial_key.source}, #{locals_key.source}")
+              end
             end
           end
         end

data/lib/rubocop-github-rails.rb

@@ -6,12 +6,10 @@ require "rubocop/github/inject"
 
 RuboCop::GitHub::Inject.rails_defaults!
 
-require "rubocop/cop/github/rails_application_record"
 require "rubocop/cop/github/rails_controller_render_action_symbol"
 require "rubocop/cop/github/rails_controller_render_literal"
 require "rubocop/cop/github/rails_controller_render_paths_exist"
 require "rubocop/cop/github/rails_controller_render_shorthand"
-require "rubocop/cop/github/rails_render_inline"
 require "rubocop/cop/github/rails_render_object_collection"
 require "rubocop/cop/github/rails_view_render_literal"
 require "rubocop/cop/github/rails_view_render_paths_exist"

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: rubocop-github
 version: !ruby/object:Gem::Version
-  version: 0.19.0
+  version: 0.20.0
 platform: ruby
 authors:
 - GitHub
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2022-10-13 00:00:00.000000000 Z
+date: 2022-10-28 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rubocop
@@ -16,42 +16,42 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 1.0.0
+        version: '1.37'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 1.0.0
+        version: '1.37'
 - !ruby/object:Gem::Dependency
   name: rubocop-performance
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '1.15'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '1.15'
 - !ruby/object:Gem::Dependency
   name: rubocop-rails
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '2.17'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '2.17'
 - !ruby/object:Gem::Dependency
   name: actionview
   requirement: !ruby/object:Gem::Requirement
@@ -105,21 +105,20 @@ files:
 - STYLEGUIDE.md
 - config/default.yml
 - config/default_cops.yml
+- config/default_pending.yml
 - config/rails.yml
 - config/rails_cops.yml
+- config/rails_pending.yml
 - guides/rails-controller-render-shorthand.md
-- guides/rails-render-inline.md
 - guides/rails-render-literal.md
 - lib/rubocop-github-rails.rb
 - lib/rubocop-github.rb
 - lib/rubocop/cop/github.rb
 - lib/rubocop/cop/github/insecure_hash_algorithm.rb
-- lib/rubocop/cop/github/rails_application_record.rb
 - lib/rubocop/cop/github/rails_controller_render_action_symbol.rb
 - lib/rubocop/cop/github/rails_controller_render_literal.rb
 - lib/rubocop/cop/github/rails_controller_render_paths_exist.rb
 - lib/rubocop/cop/github/rails_controller_render_shorthand.rb
-- lib/rubocop/cop/github/rails_render_inline.rb
 - lib/rubocop/cop/github/rails_render_object_collection.rb
 - lib/rubocop/cop/github/rails_view_render_literal.rb
 - lib/rubocop/cop/github/rails_view_render_paths_exist.rb

data/guides/rails-render-inline.md

@@ -1,27 +0,0 @@
-# GitHub/RailsRenderInline
-
-tldr; Do not use `render inline:`.
-
-## Rendering plain text
-
-``` ruby
-render inline: "Just plain text" # bad
-```
-
-The `inline:` option is often misused when plain text is being returned. Instead use `render plain: "Just plain text"`.
-
-## Rendering a dynamic ERB string
-
-String `#{}` interpolation is often misused with `render inline:` instead of using `<%= %>` interpolation. This will lead to a memory leak where an ERB method will be compiled and cached for each invocation of `render inline:`.
-
-``` ruby
-render inline: "Hello #{@name}" # bad
-```
-
-## Rendering static ERB strings
-
-``` ruby
-render inline: "Hello <%= @name %>" # bad
-```
-
-If you are passing a static ERB string to `render inline:`, this string is best moved to a `.erb` template under `app/views`. Template files are able to be precompiled at boot time.

data/lib/rubocop/cop/github/rails_application_record.rb

@@ -1,29 +0,0 @@
-# frozen_string_literal: true
-
-require "rubocop"
-
-module RuboCop
-  module Cop
-    module GitHub
-      class RailsApplicationRecord < Base
-        MSG = "Models should subclass from ApplicationRecord"
-
-        def_node_matcher :active_record_base_const?, <<-PATTERN
-          (const (const nil? :ActiveRecord) :Base)
-        PATTERN
-
-        def_node_matcher :application_record_const?, <<-PATTERN
-          (const nil? :ApplicationRecord)
-        PATTERN
-
-        def on_class(node)
-          klass, superclass, _ = *node
-
-          if active_record_base_const?(superclass) && !(application_record_const?(klass))
-            add_offense(superclass)
-          end
-        end
-      end
-    end
-  end
-end

data/lib/rubocop/cop/github/rails_render_inline.rb

@@ -1,29 +0,0 @@
-# frozen_string_literal: true
-
-require "rubocop"
-
-module RuboCop
-  module Cop
-    module GitHub
-      class RailsRenderInline < Base
-        MSG = "Avoid `render inline:`"
-
-        def_node_matcher :render_with_options?, <<-PATTERN
-          (send nil? {:render :render_to_string} (hash $...))
-        PATTERN
-
-        def_node_matcher :inline_key?, <<-PATTERN
-          (pair (sym :inline) $_)
-        PATTERN
-
-        def on_send(node)
-          if option_pairs = render_with_options?(node)
-            if option_pairs.detect { |pair| inline_key?(pair) }
-              add_offense(node)
-            end
-          end
-        end
-      end
-    end
-  end
-end