diff --git a/lib/vagrant/plugin/v2/trigger.rb b/lib/vagrant/plugin/v2/trigger.rb
index a42887f6b..4fe0d038e 100644
--- a/lib/vagrant/plugin/v2/trigger.rb
+++ b/lib/vagrant/plugin/v2/trigger.rb
@@ -109,7 +109,6 @@ module Vagrant
           triggers.each do |trigger|
             @logger.debug("Running trigger #{trigger.id}...")
 
-            # TODO: I18n me
             if !trigger.name.nil?
               @machine.ui.info(I18n.t("vagrant.trigger.fire_with_name",
                                       name: trigger.name))
diff --git a/plugins/kernel_v2/config/vm_trigger.rb b/plugins/kernel_v2/config/vm_trigger.rb
index 8f71ec5a3..9fa7ebba9 100644
--- a/plugins/kernel_v2/config/vm_trigger.rb
+++ b/plugins/kernel_v2/config/vm_trigger.rb
@@ -144,8 +144,6 @@ module VagrantPlugins
                                 cmd: @command))
         end
 
-        # TODO: Does it make sense to strip out the "shell provisioner" error key here?
-        #       We could add more context around triggers?
         if @run
           errorz = @run.validate(machine)
           errors.concat errorz["shell provisioner"] if !errorz.empty?
@@ -179,13 +177,6 @@ module VagrantPlugins
           end
         end
 
-        if !@only_on.nil?
-          # this check isn't quite right...
-          #if @only_on.all? { |o| !o.is_a?(String) || !o.is_a?(Regexp) }
-          #    errors << I18n.t("vagrant.config.triggers.only_on_bad_type")
-          #end
-        end
-
         errors
       end