You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I'm going to disable these to get the builds green for now, but we should investigate these warnings and see what's necessary to resolve them:
Offenses:
Error: app/controllers/api/v1/cookbook_versions_controller.rb:26:21: C: Rails/SkipsModelValidations: Avoid using increment_counter because it skips validations.
CookbookVersion.increment_counter(:api_download_count, @cookbook_version.id)
^^^^^^^^^^^^^^^^^
Error: app/controllers/api/v1/cookbook_versions_controller.rb:27:14: C: Rails/SkipsModelValidations: Avoid using increment_counter because it skips validations.
Cookbook.increment_counter(:api_download_count, @cookbook.id)
^^^^^^^^^^^^^^^^^
Error: app/controllers/cookbook_versions_controller.rb:10:21: C: Rails/SkipsModelValidations: Avoid using increment_counter because it skips validations.
CookbookVersion.increment_counter(:web_download_count, @version.id)
^^^^^^^^^^^^^^^^^
Error: app/controllers/cookbook_versions_controller.rb:11:14: C: Rails/SkipsModelValidations: Avoid using increment_counter because it skips validations.
Cookbook.increment_counter(:web_download_count, @cookbook.id)
^^^^^^^^^^^^^^^^^
Error: spec/controllers/api/v1/cookbooks_controller_spec.rb:25:20: C: Rails/SkipsModelValidations: Avoid using touch because it skips validations.
slow_cooking.touch
^^^^^
Error: spec/controllers/cookbooks_controller_spec.rb:48:20: C: Rails/SkipsModelValidations: Avoid using touch because it skips validations.
cookbook_2.touch
^^^^^
Error: spec/models/cookbook_spec.rb:734:13: C: Rails/SkipsModelValidations: Avoid using touch because it skips validations.
great.touch
^^^^^
383 files inspected, 7 offenses detected
Error: Process completed with exit code 1.
The text was updated successfully, but these errors were encountered:
I'm going to disable these to get the builds green for now, but we should investigate these warnings and see what's necessary to resolve them:
The text was updated successfully, but these errors were encountered: