Fix lint rules

client/python/cli/command/catalogs.py

@@ -16,15 +16,15 @@
 # specific language governing permissions and limitations
 # under the License.
 #
-from dataclasses import dataclass, field
-from typing import Dict, Optional, List
+from dataclasses import dataclass
+from typing import Dict, List
 
 from pydantic import StrictStr
 
 from cli.command import Command
 from cli.constants import StorageType, CatalogType, Subcommands, Arguments
 from cli.options.option_tree import Argument
-from polaris.management import PolarisDefaultApi, Catalog, CreateCatalogRequest, UpdateCatalogRequest, \
+from polaris.management import PolarisDefaultApi, CreateCatalogRequest, UpdateCatalogRequest, \
     StorageConfigInfo, ExternalCatalog, AwsStorageConfigInfo, AzureStorageConfigInfo, GcpStorageConfigInfo, \
     PolarisCatalog, CatalogProperties
 

client/python/cli/command/namespaces.py

@@ -27,7 +27,6 @@
 from cli.constants import Subcommands, Arguments, UNIT_SEPARATOR
 from cli.options.option_tree import Argument
 from polaris.catalog import IcebergCatalogAPI, CreateNamespaceRequest, ApiClient, Configuration
-from polaris.catalog.exceptions import NotFoundException
 from polaris.management import PolarisDefaultApi
 
 

client/python/cli/command/principal_roles.py

@@ -25,7 +25,7 @@
 from cli.constants import Subcommands, Arguments
 from cli.options.option_tree import Argument
 from polaris.management import PolarisDefaultApi, CreatePrincipalRoleRequest, PrincipalRole, UpdatePrincipalRoleRequest, \
-    GrantCatalogRoleRequest, CatalogRole, GrantPrincipalRoleRequest
+    GrantPrincipalRoleRequest
 
 
 @dataclass

client/python/cli/command/profiles.py

@@ -22,7 +22,6 @@
 from dataclasses import dataclass
 from typing import Dict, Optional, List
 
-from pydantic import StrictStr
 
 from cli.command import Command
 from cli.constants import Subcommands, DEFAULT_HOSTNAME, DEFAULT_PORT, CONFIG_DIR, CONFIG_FILE

client/python/cli/options/parser.py

@@ -68,7 +68,7 @@ def add_arguments(parser, args: List[Argument]):
                 if arg.default:
                     kwargs['default'] = arg.default
 
-                if arg.type == bool:
+                if arg.type is bool:
                     del kwargs['type']
                     parser.add_argument(arg.get_flag_name(), **kwargs, action='store_true')
                 elif arg.allow_repeats:
@@ -128,7 +128,7 @@ def parse_args(self, args=None, namespace=None):
             tree_str = self._get_tree_str(args[:help_index])
             if tree_str:
                 print(f'input: polaris {" ".join(args)}')
-                print(f'options:')
+                print('options:')
                 print(tree_str)
                 print('\n')
                 self.print_usage()
@@ -187,7 +187,7 @@ def _get_command_path(self, args: List[str], options: List[Option]) -> List[str]
                     parser = parser._subparsers._group_actions[0].choices.get(arg)
                     if not parser:
                         break
-                except Exception as e:
+                except Exception:
                     break
                 options = list(filter(lambda o: o.name == arg, options))[0].children
                 if options is None:

client/python/cli/polaris_cli.py

@@ -133,7 +133,7 @@ def _get_client_builder(options):
             if options.host is not None or options.port is not None:
                 raise Exception(f'Please provide either {Argument.to_flag_name(Arguments.BASE_URL)} or'
                                 f' {Argument.to_flag_name(Arguments.HOST)} &'
-                                f' {Argument.to_flag_name(Arguments.PORT)}, but not both');
+                                f' {Argument.to_flag_name(Arguments.PORT)}, but not both')
 
             polaris_management_url = f'{options.base_url}/api/management/v1'
             polaris_catalog_url = f'{options.base_url}/api/catalog/v1'