-
-
Save bogdanRada/c389bc7a39dc8101b21effe6d3fed21b to your computer and use it in GitHub Desktop.
Basecamp rubocop config inspiration. This template is inspired on https://gist.githubusercontent.com/lazaronixon/7815d84702f277ead5e89c9f2aa5581f/raw/rubocop.yml
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
# This template is inspired on https://gist.githubusercontent.com/lazaronixon/7815d84702f277ead5e89c9f2aa5581f/raw/rubocop.yml | |
require: | |
- rubocop-performance | |
- rubocop-rails | |
- rubocop-rspec | |
Rails: | |
Enabled: true | |
AllCops: | |
TargetRubyVersion: 3.1 | |
# RuboCop has a bunch of cops enabled by default. This setting tells RuboCop | |
# to ignore them, so only the ones explicitly set in this file are enabled. | |
DisabledByDefault: true | |
SuggestExtensions: false | |
Exclude: | |
- '**/tmp/**/*' | |
- '**/templates/**/*' | |
- '**/vendor/**/*' | |
- '**/node_modules/**/*' | |
- 'db/schema.rb' | |
Performance: | |
Exclude: | |
- 'test/**/*' | |
- 'spec/**/*' | |
# Prefer assert_not over assert ! | |
Rails/AssertNot: | |
Include: | |
- '**/test/**/*' | |
- '**/spec/**/*' | |
# Prefer assert_not_x over refute_x | |
Rails/RefuteMethods: | |
Include: | |
- '**/test/**/*' | |
- '**/spec/**/*' | |
Rails/IndexBy: | |
Enabled: true | |
Rails/IndexWith: | |
Enabled: true | |
Rails/ActionControllerFlashBeforeRender: | |
Enabled: true | |
Rails/ActionControllerTestCase: | |
Enabled: true | |
Rails/ActiveRecordCallbacksOrder: | |
Enabled: true | |
Rails/ActiveSupportOnLoad: | |
Enabled: true | |
Rails/AddColumnIndex: | |
Enabled: true | |
Rails/AfterCommitOverride: | |
Enabled: true | |
Rails/AttributeDefaultBlockValue: | |
Enabled: true | |
Rails/CompactBlank: | |
Enabled: true | |
Rails/DeprecatedActiveModelErrorsMethods: | |
Enabled: true | |
Rails/DotSeparatedKeys: | |
Enabled: true | |
Rails/DuplicateAssociation: | |
Enabled: true | |
Rails/DuplicateScope: | |
Enabled: true | |
Rails/DurationArithmetic: | |
Enabled: true | |
Rails/EagerEvaluationLogMessage: | |
Enabled: true | |
Rails/ExpandedDateRange: | |
Enabled: true | |
Rails/FindById: | |
Enabled: true | |
Rails/FreezeTime: | |
Enabled: true | |
Rails/I18nLazyLookup: | |
Enabled: true | |
Rails/I18nLocaleAssignment: | |
Enabled: true | |
Rails/I18nLocaleTexts: | |
Enabled: true | |
Rails/Inquiry: | |
Enabled: true | |
Rails/MailerName: | |
Enabled: true | |
Rails/MatchRoute: | |
Enabled: true | |
Rails/MigrationClassName: | |
Enabled: true | |
Rails/NegateInclude: | |
Enabled: true | |
Rails/Pluck: | |
Enabled: true | |
Rails/PluckInWhere: | |
Enabled: true | |
Rails/RedundantPresenceValidationOnBelongsTo: | |
Enabled: true | |
Rails/RedundantTravelBack: | |
Enabled: true | |
Rails/RenderInline: | |
Enabled: true | |
Rails/RenderPlainText: | |
Enabled: true | |
Rails/RootJoinChain: | |
Enabled: true | |
Rails/RootPathnameMethods: | |
Enabled: true | |
Rails/RootPublicPath: | |
Enabled: true | |
Rails/ShortI18n: | |
Enabled: true | |
Rails/SquishedSQLHeredocs: | |
Enabled: true | |
Rails/StripHeredoc: | |
Enabled: true | |
Rails/TimeZoneAssignment: | |
Enabled: true | |
Rails/ToFormattedS: | |
Enabled: true | |
Rails/ToSWithArgument: | |
Enabled: true | |
Rails/TopLevelHashWithIndifferentAccess: | |
Enabled: true | |
Rails/TransactionExitStatement: | |
Enabled: true | |
Rails/UnusedIgnoredColumns: | |
Enabled: true | |
Rails/WhereEquals: | |
Enabled: true | |
Rails/WhereExists: | |
Enabled: true | |
Rails/WhereMissing: | |
Enabled: true | |
Rails/WhereNot: | |
Enabled: true | |
# Prefer &&/|| over and/or. | |
Style/AndOr: | |
Enabled: true | |
# Align `when` with `case`. | |
Layout/CaseIndentation: | |
Enabled: true | |
Layout/ClosingHeredocIndentation: | |
Enabled: true | |
Layout/ClosingParenthesisIndentation: | |
Enabled: true | |
# Align comments with method definitions. | |
Layout/CommentIndentation: | |
Enabled: true | |
Layout/ElseAlignment: | |
Enabled: true | |
# Align `end` with the matching keyword or starting expression except for | |
# assignments, where it should be aligned with the LHS. | |
Layout/EndAlignment: | |
Enabled: true | |
EnforcedStyleAlignWith: variable | |
AutoCorrect: true | |
Layout/EndOfLine: | |
Enabled: true | |
Layout/EmptyLineAfterMagicComment: | |
Enabled: true | |
Layout/EmptyLinesAroundAccessModifier: | |
Enabled: true | |
EnforcedStyle: only_before | |
Layout/EmptyLinesAroundBlockBody: | |
Enabled: true | |
# In a regular class definition, no empty lines around the body. | |
Layout/EmptyLinesAroundClassBody: | |
Enabled: true | |
# In a regular method definition, no empty lines around the body. | |
Layout/EmptyLinesAroundMethodBody: | |
Enabled: true | |
# In a regular module definition, no empty lines around the body. | |
Layout/EmptyLinesAroundModuleBody: | |
Enabled: true | |
# Use Ruby >= 1.9 syntax for hashes. Prefer { a: :b } over { :a => :b }. | |
Style/HashSyntax: | |
Enabled: true | |
EnforcedShorthandSyntax: never | |
# Method definitions after `private` or `protected` isolated calls need one | |
# extra level of indentation. | |
Layout/IndentationConsistency: | |
Enabled: true | |
EnforcedStyle: indented_internal_methods | |
# Two spaces, no tabs (for indentation). | |
Layout/IndentationWidth: | |
Enabled: true | |
Layout/LeadingCommentSpace: | |
Enabled: true | |
Layout/SpaceAfterColon: | |
Enabled: true | |
Layout/SpaceAfterComma: | |
Enabled: true | |
Layout/SpaceAfterSemicolon: | |
Enabled: true | |
Layout/SpaceAroundEqualsInParameterDefault: | |
Enabled: true | |
Layout/SpaceAroundKeyword: | |
Enabled: true | |
Layout/SpaceAroundOperators: | |
Enabled: true | |
Layout/SpaceBeforeComma: | |
Enabled: true | |
Layout/SpaceBeforeComment: | |
Enabled: true | |
Layout/SpaceBeforeFirstArg: | |
Enabled: true | |
Style/DefWithParentheses: | |
Enabled: true | |
# Defining a method with parameters needs parentheses. | |
Style/MethodDefParentheses: | |
Enabled: true | |
Style/ExplicitBlockArgument: | |
Enabled: true | |
Style/FrozenStringLiteralComment: | |
Enabled: false | |
EnforcedStyle: always | |
Style/MapToHash: | |
Enabled: true | |
Style/RedundantFreeze: | |
Enabled: true | |
# Use `foo {}` not `foo{}`. | |
Layout/SpaceBeforeBlockBraces: | |
Enabled: true | |
# Use `foo { bar }` not `foo {bar}`. | |
Layout/SpaceInsideBlockBraces: | |
Enabled: true | |
EnforcedStyleForEmptyBraces: space | |
# Use `{ a: 1 }` not `{a:1}`. | |
Layout/SpaceInsideHashLiteralBraces: | |
Enabled: true | |
Layout/SpaceInsideParens: | |
Enabled: true | |
# Check quotes usage according to lint rule below. | |
Style/StringLiterals: | |
Enabled: true | |
EnforcedStyle: double_quotes | |
Include: | |
- 'app/**/*' | |
- 'test/**/*' | |
- 'spec/**/*' | |
# Detect hard tabs, no hard tabs. | |
Layout/IndentationStyle: | |
Enabled: true | |
# Empty lines should not have any spaces. | |
Layout/TrailingEmptyLines: | |
Enabled: true | |
# No trailing whitespace. | |
Layout/TrailingWhitespace: | |
Enabled: true | |
# Use quotes for string literals when they are enough. | |
Style/RedundantPercentQ: | |
Enabled: true | |
Lint/AmbiguousOperator: | |
Enabled: true | |
Lint/AmbiguousRegexpLiteral: | |
Enabled: true | |
Lint/DuplicateRequire: | |
Enabled: true | |
Lint/DuplicateMethods: | |
Enabled: true | |
Lint/ErbNewArguments: | |
Enabled: true | |
Lint/EnsureReturn: | |
Enabled: true | |
# Use my_method(my_arg) not my_method( my_arg ) or my_method my_arg. | |
Lint/RequireParentheses: | |
Enabled: true | |
Lint/RedundantStringCoercion: | |
Enabled: true | |
Lint/UriEscapeUnescape: | |
Enabled: true | |
Lint/UselessAssignment: | |
Enabled: true | |
Lint/DeprecatedClassMethods: | |
Enabled: true | |
Style/ParenthesesAroundCondition: | |
Enabled: true | |
Style/HashTransformKeys: | |
Enabled: true | |
Style/HashTransformValues: | |
Enabled: true | |
Style/RedundantBegin: | |
Enabled: true | |
Style/RedundantReturn: | |
Enabled: true | |
AllowMultipleReturnValues: true | |
Style/RedundantRegexpEscape: | |
Enabled: true | |
Style/Semicolon: | |
Enabled: true | |
AllowAsExpressionSeparator: true | |
# Prefer Foo.method over Foo::method | |
Style/ColonMethodCall: | |
Enabled: true | |
Style/TrivialAccessors: | |
Enabled: true | |
##### PERFORMANCE ##### | |
Performance/BindCall: | |
Enabled: true | |
Performance/FlatMap: | |
Enabled: true | |
Performance/MapCompact: | |
Enabled: true | |
Performance/SelectMap: | |
Enabled: true | |
Performance/RedundantMerge: | |
Enabled: true | |
Performance/StartWith: | |
Enabled: true | |
Performance/EndWith: | |
Enabled: true | |
Performance/RegexpMatch: | |
Enabled: true | |
Performance/ReverseEach: | |
Enabled: true | |
Performance/StringReplacement: | |
Enabled: true | |
Performance/UnfreezeString: | |
Enabled: true | |
Performance/DeletePrefix: | |
Enabled: true | |
Performance/DeleteSuffix: | |
Enabled: true | |
Performance/OpenStruct: | |
Enabled: true | |
RSpec/MultipleExpectations: | |
Enabled: true | |
Max: 5 | |
RSpec/NestedGroups: | |
Enabled: false |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment