Handle a lot of TODOs

lib/vagrant/action/builtin/box_add.rb

@@ -204,7 +204,6 @@ module Vagrant
                 url: url,
                 versions: metadata.versions.join(", ")
             else
-              # TODO: show supported providers
               raise Errors::BoxAddNoMatchingProvider,
                 name: metadata.name,
                 requested: provider,

lib/vagrant/action/runner.rb

@@ -3,9 +3,6 @@ require 'log4r'
 require 'vagrant/action/hook'
 require 'vagrant/util/busy'
 
-# TODO:
-# * env.lock
-
 module Vagrant
   module Action
     class Runner

lib/vagrant/config/loader.rb

@@ -76,7 +76,6 @@ module Vagrant
 
         unknown_sources = @sources.keys - order
         if !unknown_sources.empty?
-          # TODO: Raise exception here perhaps.
           @logger.error("Unknown config sources: #{unknown_sources.inspect}")
         end
 

lib/vagrant/util/template_renderer.rb

@@ -56,7 +56,6 @@ module Vagrant
       #
       # @return [String]
       def render
-        # TODO: Seems like a pretty dirty way to do this. Perhaps refactor this
         old_template = template
         result = nil
         File.open(full_template_path, 'r') do |f|

plugins/kernel_v1/config/ssh.rb

@@ -38,8 +38,6 @@ module VagrantPlugins
         new.ssh.forward_agent    = @forward_agent if @forward_agent != UNSET_VALUE
         new.ssh.forward_x11      = @forward_x11 if @forward_x11 != UNSET_VALUE
         new.ssh.shell            = @shell if @shell != UNSET_VALUE
-
-        # TODO: Warn that max_tries and timeout are gone
       end
     end
   end

plugins/providers/docker/driver.rb

@@ -19,7 +19,9 @@ module VagrantPlugins
         regexp = /Successfully built (.+)$/i
         match  = regexp.match(result)
         if !match
-          # TODO: error
+          # This will cause a stack trace in Vagrant, but it is a bug
+          # if this happens anyways.
+          raise "UNKNOWN OUTPUT: #{result}"
         end
 
         match[1]
@@ -86,7 +88,6 @@ module VagrantPlugins
           execute('docker', 'start', cid)
           # This resets the cached information we have around, allowing `vagrant reload`s
           # to work properly
-          # TODO: Add spec to verify this behavior
           @data = nil
         end
       end