Appease the dog overlords

app/controllers/notifications_controller.rb

@@ -8,7 +8,7 @@ class NotificationsController < ApplicationController
     "comment"      => Notification::Commented.name,
     "commentsmile" => Notification::CommentSmiled.name,
     "relationship" => Notification::StartedFollowing.name,
-    "smile"        => Notification::Smiled.name,
+    "smile"        => Notification::Smiled.name
   }.freeze
 
   def index

app/models/notification.rb

@@ -1,5 +1,7 @@
+# frozen_string_literal: true
+
 class Notification < ApplicationRecord
-  belongs_to :recipient, class_name: 'User'
+  belongs_to :recipient, class_name: "User"
   belongs_to :target, polymorphic: true
 
   class << self
@@ -8,12 +10,12 @@ class Notification < ApplicationRecord
     define_cursor_paginator :cursored_for, :for
     define_cursor_paginator :cursored_for_type, :for_type
 
-    def for(recipient, options={})
+    def for(recipient, options = {})
-      self.where(options.merge!(recipient: recipient)).order(:created_at).reverse_order
+      where(options.merge!(recipient: recipient)).order(:created_at).reverse_order
     end
 
-    def for_type(recipient, type, options={})
+    def for_type(recipient, type, options = {})
-      self.where(options.merge!(recipient: recipient)).where(type: type).order(:created_at).reverse_order
+      where(options.merge!(recipient: recipient)).where(type: type).order(:created_at).reverse_order
     end
 
     def notify(recipient, target)
@@ -33,7 +35,7 @@ class Notification < ApplicationRecord
       return nil unless notif_type
 
       notif = Notification.find_by(recipient: recipient, target: target)
-      notif.destroy unless notif.nil?
+      notif&.destroy
     end
 
     private

app/models/notification/comment_smiled.rb

@@ -1,2 +1,4 @@
+# frozen_string_literal: true
+
 class Notification::CommentSmiled < Notification
 end

app/models/notification/commented.rb

@@ -1,2 +1,4 @@
+# frozen_string_literal: true
+
 class Notification::Commented < Notification
 end

app/models/notification/question_answered.rb

@@ -1,2 +1,4 @@
+# frozen_string_literal: true
+
 class Notification::QuestionAnswered < Notification
 end

app/models/notification/smiled.rb

@@ -1,2 +1,4 @@
+# frozen_string_literal: true
+
 class Notification::Smiled < Notification
 end

app/models/notification/started_following.rb

@@ -1,2 +1,4 @@
+# frozen_string_literal: true
+
 class Notification::StartedFollowing < Notification
 end