refacto: remove todo

Author: sfauvel

mithril-aggregator/src/commands/mod.rs

@@ -103,7 +103,6 @@ impl Source for MainOpts {
         let mut result = Map::new();
         let namespace = "clap arguments".to_string();
 
-        // TODO Is it normal to only have db_directory ?
         register_config_value_option!(result, &namespace, self.db_directory, |v: PathBuf| format!(
             "{}",
             v.to_string_lossy()

mithril-aggregator/src/commands/serve_command.rs

@@ -99,7 +99,6 @@ impl Source for ServeCommand {
         register_config_value_bool!(result, &namespace, self.enable_metrics_server);
         register_config_value_option!(result, &namespace, self.metrics_server_ip);
         register_config_value_option!(result, &namespace, self.metrics_server_port);
-        // TODO is it normal to pass a Some(v) and not only v when value is present ?
         register_config_value_option!(
             result,
             &namespace,