This repository has been archived by the owner on May 2, 2024. It is now read-only.
Linting fixes #4
reesericciAnnotations
13 errors and 1 warning
|
app/controllers/domains_controller.rb#L63
Use `save!` instead of `save` if the return value is not checked.
|
|
app/controllers/records_controller.rb#L14
Use `create!` instead of `create` if the return value is not checked.
|
|
app/controllers/records_controller.rb#L70
Use `save!` instead of `save` if the return value is not checked.
|
|
app/controllers/users_controller.rb#L10
Prefer `!User::User.find_by(email: params[:email]).nil?` over `User::User.find_by(email: params[:email]) != nil`.
|
|
app/controllers/users_controller.rb#L49
Use `update!` instead of `update` if the return value is not checked.
|
|
app/controllers/users_controller.rb#L76
Use `save!` instead of `save` if the return value is not checked.
|
|
app/models/domain.rb#L3
Uniqueness validation should have a unique index on the database column.
|
|
app/models/user/user.rb#L2
Uniqueness validation should have a unique index on the database column.
|
|
app/models/user/user.rb#L3
Specify a `:dependent` option.
|
|
app/models/user/user.rb#L13
Use `save!` instead of `save` if the return value is not checked.
|
|
app/models/user/user.rb#L19
Prefer `!begin
@hotp.verify(token.to_s, self.otp_counter.to_i)
rescue
nil
end.nil?` over `begin
@hotp.verify(token.to_s, self.otp_counter.to_i)
rescue
nil
end != nil`.
|
|
config/environments/production.rb#L55
Prefer `to_s` over string interpolation.
|
|
db/migrate/20230606000652_create_new_user_model_and_delete_old_user_model.rb#L4
drop_table(without block) is not reversible.
|
|
db/migrate/20231222071308_add_service_name_to_active_storage_blobs.active_storage.rb#L9
Wrap assignment in parentheses if intentional
|