Skip to content

Commit 899ace2

Browse files
committed
Fix linting issues
1 parent d5b57db commit 899ace2

File tree

2 files changed

+9
-5
lines changed

2 files changed

+9
-5
lines changed

vecorel_cli/cli/logger.py

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
import re
22
import sys
33
from logging import Logger
4-
from typing import Any, Optional
4+
from typing import Any, Optional, Union
55

66
from loguru import logger
77

@@ -36,7 +36,7 @@ def info(self, message: str, **kwargs):
3636
def warning(self, message: str, **kwargs):
3737
self.log(message, "warning", **kwargs)
3838

39-
def error(self, message: str, **kwargs):
39+
def error(self, message: Union[Exception, str], **kwargs):
4040
self.log(message, "error", **kwargs)
4141

4242
def success(self, message: str, **kwargs):
@@ -58,7 +58,9 @@ def log(
5858
# Escape XML/HTML tags etc. that look like loguru color directives
5959
# The regexp is coming directly from the loguru source code, see
6060
# https://github.com/Delgan/loguru/blob/master/loguru/_colorizer.py
61-
message = re.sub(r"(</?(?:[fb]g\s)?[^<>\s]*>)", r"\\\1", message, count=0, flags=re.IGNORECASE)
61+
message = re.sub(
62+
r"(</?(?:[fb]g\s)?[^<>\s]*>)", r"\\\1", message, count=0, flags=re.IGNORECASE
63+
)
6264

6365
# Handle indentation (including multiple lines)
6466
message = self._indent_text(message, indent)

vecorel_cli/validate.py

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -35,14 +35,14 @@ def get_cli_args():
3535
def validate_cli(
3636
self,
3737
source: list[Union[str, Path]],
38-
num: int = 100,
38+
num: Optional[int] = 100,
3939
schema_map: SchemaMapping = {},
4040
):
4141
if not isinstance(source, list):
4242
raise ValueError("Source must be a list.")
4343
if len(source) == 0:
4444
raise ValueError("No source files provided")
45-
if num < 0:
45+
if num is not None and num < 0:
4646
num = None
4747

4848
invalid = 0
@@ -102,6 +102,8 @@ def validate(
102102
) -> Validator:
103103
encoding = create_encoding(file)
104104
validator = encoding.get_validator()
105+
if validator is None:
106+
raise ValueError(f"No validator available for files of type {encoding.get_format()}")
105107
validator.set_required_schemas(Registry.required_extensions)
106108
validator.validate(num=num, schema_map=schema_map)
107109
return validator

0 commit comments

Comments
 (0)