Merge #674

674: Fix pylint and isort errors r=alallema a=sanders41

# Pull Request

## Related issue
N/A

## What does this PR do?
- Fixes the pylint errors in #672
- Fixes isort upgrade errors in #667
- Causes headaches from fighting with pipenv again 😄 

## PR checklist
Please check if your PR fulfills the following requirements:
- [x] Does this PR fix an existing issue, or have you listed the changes applied in the PR description (and why they are needed)?
- [x] Have you read the contributing guidelines?
- [x] Have you made sure that the title is accurate and descriptive of the changes?

Thank you so much for contributing to Meilisearch!


Co-authored-by: Paul Sanders <[email protected]>

Author: bors[bot]

Pipfile

@@ -15,7 +15,7 @@ types-requests = "*"
 black = "*"
 isort = "*"
 importlib_metadata = {version = "*", markers="python_version < '3.8'"}
-zipp = {version = "*", markers="python_version < '3.8'"}
+zipp = {version = "==3.8.1", markers="python_version < '3.8'"}
 exceptiongroup = {version = "*", markers="python_version < '3.11'"}
 tomli = {version = "*", markers="python_version < '3.11'"}
 wrapt = {version = "*", markers="python_version < '3.11'"}

Pipfile.lock

@@ -198,7 +198,7 @@ def index(self, uid: str) -> Index:
         """
         if uid is not None:
             return Index(self.config, uid=uid)
-        raise Exception("The index UID should not be None")
+        raise ValueError("The index UID should not be None")
 
     def get_all_stats(self) -> Dict[str, Any]:
         """Get all stats of Meilisearch
@@ -566,15 +566,15 @@ def generate_tenant_token(
         """
         # Validate all fields
         if api_key == "" or api_key is None and self.config.api_key is None:
-            raise Exception(
+            raise ValueError(
                 "An api key is required in the client or should be passed as an argument."
             )
         if api_key_uid == "" or api_key_uid is None or self._valid_uuid(api_key_uid) is False:
-            raise Exception("An uid is required and must comply to the uuid4 format.")
+            raise ValueError("An uid is required and must comply to the uuid4 format.")
         if not search_rules or search_rules == [""]:
-            raise Exception("The search_rules field is mandatory and should be defined.")
+            raise ValueError("The search_rules field is mandatory and should be defined.")
         if expires_at and expires_at < datetime.datetime.utcnow():
-            raise Exception("The date expires_at should be in the future.")
+            raise ValueError("The date expires_at should be in the future.")
 
         # Standard JWT header for encryption with SHA256/HS256 algorithm
         header = {"typ": "JWT", "alg": "HS256"}