delta/gitlab/gitlab-ce.git/rubocop/cop, branch fix-github-clone-wiki

Added Cop to blacklist polymorphic associations

2017-06-07T15:36:55+00:00

One should really use a separate table instead of using polymorphic
associations.

See https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/11168 for
more information.

Add a rubocop rule to check if a method 'redirect_to' is used without explicitly set 'status' in 'destroy' actions of controllers

2017-06-06T22:45:16+00:00

Added Cop to blacklist the use of serialize

2017-05-31T12:03:37+00:00

This Cop blacklists the use of ActiveRecord's "serialize" method, except
for cases where we already use this.

New Migration/UpdateColumnInBatches cop

2017-05-29T09:28:23+00:00

Signed-off-by: Rémy Coutable

Add AddColumnWithDefaultToLargeTable cop

2017-04-28T20:55:55+00:00

Rename AddColumnWithDefault to ReversibleAddColumnWithDefault

2017-04-28T20:55:53+00:00

We're going to add another cop that deals with another aspect of
`add_column_with_default`, so we need to separate them.

Refactor the AddColumnWithDefault cop to use node matchers

2017-04-28T20:55:53+00:00

Add remove_concurrent_index to database helper

2017-04-05T22:53:57+00:00

Add cop to ensure reversibility of add_concurrent_index

2017-03-07T16:09:01+00:00

Add RuboCop cop for custom error classes

2017-03-01T10:53:10+00:00

From the Ruby style guide:

    # bad
    class FooError < StandardError
    end

    # okish
    class FooError < StandardError; end

    # good
    FooError = Class.new(StandardError)

This cop does that, but only for error classes (classes where the
superclass ends in 'Error'). We have empty controllers and models, which
are perfectly valid empty classes.