Age | Commit message (Collapse) | Author |
|
|
|
Resolves the `codeclimate` CI failure
|
|
This cop adds an offense when `before`, `after`, or `around` are used as
single-line blocks.
|
|
'timestamps_with_timezone'
|
|
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.
|
|
explicitly set 'status' in 'destroy' actions of controllers
|
|
This Cop blacklists the use of ActiveRecord's "serialize" method, except
for cases where we already use this.
|
|
Signed-off-by: Rémy Coutable <remy@rymai.me>
|
|
|
|
We're going to add another cop that deals with another aspect of
`add_column_with_default`, so we need to separate them.
|
|
|
|
|
|
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.
|
|
This adds a Rubocop rule to enforce the use of
add_concurrent_foreign_key instead of the regular add_foreign_key
method. This cop has been disabled for existing migrations so we don't
need to change those.
|
|
|
|
|