Use regular params for NotificationService#close_issue

In this case we won't need to serialize/deserialize kwargs
It fixes 2.7 Ruby warnings

app/services/issues/close_service.rb

@@ -31,7 +31,7 @@ module Issues
 
         closed_via = _("commit %{commit_id}") % { commit_id: closed_via.id } if closed_via.is_a?(Commit)
 
-        notification_service.async.close_issue(issue, current_user, closed_via: closed_via) if notifications
+        notification_service.async.close_issue(issue, current_user, { closed_via: closed_via }) if notifications
         todo_service.close_issue(issue, current_user)
         resolve_alert(issue)
         execute_hooks(issue, 'close')

app/services/notification_service.rb

@@ -118,8 +118,8 @@ class NotificationService
   #  * project team members with notification level higher then Participating
   #  * users with custom level checked with "close issue"
   #
-  def close_issue(issue, current_user, closed_via: nil)
-    close_resource_email(issue, current_user, :closed_issue_email, closed_via: closed_via)
+  def close_issue(issue, current_user, params = {})
+    close_resource_email(issue, current_user, :closed_issue_email, closed_via: params[:closed_via])
   end
 
   # When we reassign an issue we should send an email to: