Escape "on" in yaml for correct parsing

TIL yaml treats true, on and yes as boolean values. So there was no
Rubocop bug, but rather `on` was not treated as a string literal.

Author: chaimann

.rubocop.yml

@@ -326,7 +326,7 @@ Naming/MethodParameterName:
     - id
     - to
     - _
-    - on
+    - "on"
 
 # Rubocop doesn't understand side-effects
 Style/IdenticalConditionalBranches:

admin/app/helpers/solidus_admin/stimulus_helper.rb

@@ -10,7 +10,7 @@ def stimulus_controller
       { "data-controller": stimulus_id }
     end
 
-    def stimulus_action(action, on: nil) # rubocop:disable Naming/MethodParameterName
+    def stimulus_action(action, on: nil)
       action_construct = []
       action_construct << "#{on}->" if on.present?
       action_construct << "#{stimulus_id}##{action}"