From 5631e22dd165a9be2d77a6566a7ae1e7c3a2fb94 Mon Sep 17 00:00:00 2001 From: Issy Long Date: Tue, 18 Oct 2022 17:47:06 +0000 Subject: [PATCH] config/rails: Broader ERB exclude paths - Yes, by default Rails has `app/views/**/*.erb`, but ERB can also be elsewhere in the codebase. - This avoids users (of the Rails configs of this gem) from having to manually add other ERB exclusions, since these cops don't reliably play nicely with any ERB regardless of where it lives. --- config/rails.yml | 20 ++++++++++---------- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/config/rails.yml b/config/rails.yml index e414a00a..3114c3cf 100644 --- a/config/rails.yml +++ b/config/rails.yml @@ -34,27 +34,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: @@ -399,16 +399,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"