changeset 113:f7bc409add34

alter some comments
author Jeff Hammel <jhammel@mozilla.com>
date Thu, 28 Jun 2012 14:48:17 -0700
parents cbd0f0aea3e5
children d1911d9b5b19
files configuration/configuration.py
diffstat 1 files changed, 3 insertions(+), 5 deletions(-) [+]
line wrap: on
line diff
--- a/configuration/configuration.py	Mon Jun 25 15:45:27 2012 -0700
+++ b/configuration/configuration.py	Thu Jun 28 14:48:17 2012 -0700
@@ -103,8 +103,9 @@
 
     configuration_providers.append(YAML())
 
-# TODO: add a configuration provider for taking command-line arguments
-# from a file
+# TODO: add configuration providers
+# - for taking command-line arguments from a file
+# - for .ini files
 
 __all__.extend([i.__class__.__name__ for i in configuration_providers])
 
@@ -185,7 +186,6 @@
         # TODO: could use 'extend'
         # - http://hg.mozilla.org/build/mozharness/file/5f44ba08f4be/mozharness/base/config.py#l41
 
-        # TODO: what about nested types?
         kw['action'] = 'append'
         return args, kw
 
@@ -222,8 +222,6 @@
             raise AssertionError("Each value must be delimited by '%s': %s" % (self.delimeter, value))
         return value.split(self.delimeter, 1)
 
-# TODO: 'dict'-type cli interface
-
 types = {bool:  BoolCLI(),
          int:   IntCLI(),
          float: FloatCLI(),