@@ -265,7 +265,6 @@ def _update_command_table_from_modules(args, command_modules=None):
265265
266266 results = self ._load_modules (args , command_modules )
267267
268- # @TODO: export to own method:
269268 count , cumulative_elapsed_time , cumulative_group_count , cumulative_command_count = \
270269 self ._process_results_with_timing (results , command_modules )
271270 # Summary line
@@ -587,25 +586,21 @@ def _handle_module_load_error(self, result):
587586 """Handle errors that occurred during module loading."""
588587 import traceback
589588 from azure .cli .core import telemetry
590-
591- # Changing this error message requires updating CI script that checks for failed module loading.
589+
592590 logger .error ("Error loading command module '%s': %s" , result .module_name , result .error )
593- telemetry .set_exception (exception = result .error ,
591+ telemetry .set_exception (exception = result .error ,
594592 fault_type = 'module-load-error-' + result .module_name ,
595593 summary = 'Error loading module: {}' .format (result .module_name ))
596594 logger .debug (traceback .format_exc ())
597595
598596 def _process_successful_load (self , result ):
599597 """Process successfully loaded module results."""
600- # Set command source for all commands in the module
601598 for cmd in result .command_table .values ():
602599 cmd .command_source = result .module_name
603-
604- # Update main command and group tables
600+
605601 self .command_table .update (result .command_table )
606602 self .command_group_table .update (result .group_table )
607-
608- # Log the results
603+
609604 logger .debug (self .item_format_string , result .module_name , result .elapsed_time ,
610605 len (result .group_table ), len (result .command_table ))
611606
0 commit comments