fix brakeman and erb lint violations

app/controllers/submitters_request_changes_controller.rb

@@ -5,7 +5,7 @@ class SubmittersRequestChangesController < ApplicationController
   skip_before_action :verify_authenticity_token, only: :request_changes
 
   def request_changes
-    if request.get?
+    if request.get? || request.head?
       render 'submitters_request_changes/request_changes', layout: false if request.xhr?
     else
       return redirect_back(fallback_location: root_path, alert: 'Invalid request') unless can_request_changes?

app/views/submissions/show.html.erb

@@ -238,8 +238,8 @@
             <% if signed_in? && submitter && submitter.completed_at? && !submitter.declined_at? && !submitter.changes_requested_at? && current_user == @submission.created_by_user %>
               <div class="mt-2 mb-1">
                 <%= link_to 'Request Changes', request_changes_submitter_path(submitter.slug),
-                    class: 'btn btn-sm btn-warning w-full',
+                            class: 'btn btn-sm btn-warning w-full',
-                    data: { turbo_frame: :modal } %>
+                            data: { turbo_frame: :modal } %>
               </div>
             <% end %>
           </div>

app/views/submitters_request_changes/request_changes.html.erb

@@ -15,11 +15,11 @@
             <span class="label-text">Message (required)</span>
           </label>
           <%= f.text_area :reason,
-              required: true,
+                          required: true,
-              class: 'textarea textarea-bordered w-full',
+                          class: 'textarea textarea-bordered w-full',
-              dir: 'auto',
+                          dir: 'auto',
-              placeholder: 'Please provide specific details about what needs to be changed...',
+                          placeholder: 'Please provide specific details about what needs to be changed...',
-              rows: '6' %>
+                          rows: '6' %>
         </div>
 
         <div class="modal-action">