Skip to content

Commit

Permalink
Merge pull request #708 from chef/trim_cops
Browse files Browse the repository at this point in the history
Turn off Naming/AccessorMethodName
  • Loading branch information
tas50 authored Aug 5, 2020
2 parents 552c6e7 + 204776f commit b740a3c
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions config/cookstyle.yml
Original file line number Diff line number Diff line change
Expand Up @@ -2342,15 +2342,11 @@ Metrics/BlockNesting:
Metrics/ParameterLists:
Enabled: false

#### COPS FOR THE NEXT MAJOR RELEASE

# This updates how we send helpers into the Chef recipe/resource classes and makes WAY more sense
# Chef::Recipe.send(:include, ::Apt::Helpers) -> Chef::Recipe.include ::Apt::Helpers
Lint/SendWithMixinArgument:
Enabled: true

#### END COPS FOR THE NEXT MAJOR RELEASE

# this migrates old # rubocop: comments to use the latest namespaces, which prevents a ton of spam during cookstyle runs
Migration/DepartmentName:
Enabled: true
Expand Down Expand Up @@ -2575,3 +2571,7 @@ Style/HashEachMethods:
# We want to catch this deprecation in cookbooks
Lint/DeprecatedOpenSSLConstant:
Enabled: true

# This just isn't a big deal in the context of a cookbook
Naming/AccessorMethodName:
Enabled: false

0 comments on commit b740a3c

Please sign in to comment.