Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bump RuboCop to 0.67.2 #35989

Merged
merged 1 commit into from Apr 16, 2019
Merged

Bump RuboCop to 0.67.2 #35989

merged 1 commit into from Apr 16, 2019

Commits on Apr 16, 2019

  1. Bump RuboCop to 0.67.2

    Performance cops will be extracted from RuboCop to RuboCop Performance
    when next RuboCop 0.68 will be released.
    rubocop/rubocop#5977
    
    RuboCop 0.67 is its transition period.
    
    Since rails/rails repository uses Performance cops, This PR added
    rubocop-performance gem to Gemfile.
    
    And this PR fixes some offenses using the following auto-correct.
    
    ```console
    % bundle exec rubocop -a
    
    Offenses:
    
    activerecord/test/cases/connection_adapters/connection_handlers_multi_db_test.rb:212:26:
    C: [Corrected] Layout/SpaceAroundOperators: Operator =
    > should be surrounded by a single space.
                  "primary"  => { adapter: "sqlite3", database: "db/primary.sqlite3" }
                             ^^
    activerecord/test/cases/connection_adapters/connection_handlers_multi_db_test.rb:239:26:
    C: [Corrected] Layout/SpaceAroundOperators: Operator => should be
    surrounded by a single space.
                  "primary"  => { adapter: "sqlite3", database: "db/primary.sqlite3" }
                             ^^
    actionview/test/template/resolver_shared_tests.rb:1:1: C: [Corrected]
    Style/FrozenStringLiteralComment: Missing magic comment #
    frozen_string_literal: true.
    module ResolverSharedTests
    ^
    actionview/test/template/resolver_shared_tests.rb:10:33: C: [Corrected]
    Layout/SpaceAroundEqualsInParameterDefault: Surrounding space missing in
    default value assignment.
      def with_file(filename, source="File at #{filename}")
                                    ^
    actionview/test/template/resolver_shared_tests.rb:106:5: C: [Corrected]
    Rails/RefuteMethods: Prefer assert_not_same over refute_same.
        refute_same a, b
        ^^^^^^^^^^^
    
    2760 files inspected, 5 offenses detected, 5 offenses corrected
    ```
    koic committed Apr 16, 2019
    Configuration menu
    Copy the full SHA
    c6379fd View commit details
    Browse the repository at this point in the history