addesssing more style issues

Author: mettursathish

src/migrate/azext_migrate/helpers/replication/get/_execute_get.py

@@ -178,7 +178,7 @@ def _format_protected_item(item):
     return formatted_item
 
 
-def _print_protected_item_details(item):
+def _print_protected_item_details(item):  # pylint: disable=too-many-nested-blocks
     """
     Print detailed information about a protected item.
 

src/migrate/azext_migrate/helpers/replication/job/_format.py

@@ -8,7 +8,7 @@
 """
 
 
-def calculate_duration(start_time, end_time):
+def calculate_duration(start_time, end_time):  # pylint: disable=too-many-return-statements
     """
     Calculate duration between two timestamps.
 

src/migrate/azext_migrate/helpers/replication/new/_validate.py

@@ -82,7 +82,7 @@ def _process_v3_dict(extended_details, app_map):
     return app_map
 
 
-def validate_server_parameters(
+def validate_server_parameters(  # pylint: disable=too-many-locals,too-many-branches
         cmd,
         machine_id,
         machine_index,

src/migrate/azext_migrate/helpers/replication/remove/_execute_delete.py

@@ -183,9 +183,9 @@ def execute_removal(cmd, subscription_id, target_object_id,
             return job_details
 
         # Job details unavailable but we have the job name
-            display_removal_success(
-                protected_item_name, job_name, resource_group_name)
-            return None
+        display_removal_success(
+            protected_item_name, job_name, resource_group_name)
+        return None
     else:
         # No job name available
         log_removal_success(protected_item_name)