chore: add Ruff logging format check
Signed-off-by: Henry Schreiner <henryschreineriii@gmail.com>
This commit is contained in:
@@ -93,7 +93,7 @@ class WindowsVersions:
|
|||||||
unsorted_versions = spec.filter(self.version_dict)
|
unsorted_versions = spec.filter(self.version_dict)
|
||||||
versions = sorted(unsorted_versions, reverse=True)
|
versions = sorted(unsorted_versions, reverse=True)
|
||||||
|
|
||||||
log.debug(f"Windows {self.arch} {spec} has {', '.join(str(v) for v in versions)}")
|
log.debug("Windows %s %s has %s", self.arch, spec, ", ".join(str(v) for v in versions))
|
||||||
|
|
||||||
if not versions:
|
if not versions:
|
||||||
return None
|
return None
|
||||||
@@ -254,7 +254,7 @@ class AllVersions:
|
|||||||
identifier = config["identifier"]
|
identifier = config["identifier"]
|
||||||
version = Version(config["version"])
|
version = Version(config["version"])
|
||||||
spec = Specifier(f"=={version.major}.{version.minor}.*")
|
spec = Specifier(f"=={version.major}.{version.minor}.*")
|
||||||
log.info(f"Reading in '{identifier}' -> {spec} @ {version}")
|
log.info("Reading in %r -> %s @ %s", str(identifier), spec, version)
|
||||||
orig_config = copy.copy(config)
|
orig_config = copy.copy(config)
|
||||||
config_update: AnyConfig | None = None
|
config_update: AnyConfig | None = None
|
||||||
|
|
||||||
@@ -282,7 +282,7 @@ class AllVersions:
|
|||||||
config.update(**config_update)
|
config.update(**config_update)
|
||||||
|
|
||||||
if config != orig_config:
|
if config != orig_config:
|
||||||
log.info(f" Updated {orig_config} to {config}")
|
log.info(" Updated %s to %s", orig_config, config)
|
||||||
|
|
||||||
|
|
||||||
@click.command()
|
@click.command()
|
||||||
|
|||||||
@@ -54,14 +54,14 @@ def git_ls_remote_versions(url) -> list[VersionTuple]:
|
|||||||
try:
|
try:
|
||||||
version = Version(version_string)
|
version = Version(version_string)
|
||||||
if version.is_devrelease:
|
if version.is_devrelease:
|
||||||
log.info(f"Ignoring development release '{version}'")
|
log.info("Ignoring development release %r", str(version))
|
||||||
continue
|
continue
|
||||||
if version.is_prerelease:
|
if version.is_prerelease:
|
||||||
log.info(f"Ignoring pre-release '{version}'")
|
log.info("Ignoring pre-release %r", str(version))
|
||||||
continue
|
continue
|
||||||
versions.append(VersionTuple(version, version_string))
|
versions.append(VersionTuple(version, version_string))
|
||||||
except InvalidVersion:
|
except InvalidVersion:
|
||||||
log.warning(f"Ignoring ref '{ref}'")
|
log.warning("Ignoring ref %r", ref)
|
||||||
versions.sort(reverse=True)
|
versions.sort(reverse=True)
|
||||||
return versions
|
return versions
|
||||||
|
|
||||||
|
|||||||
@@ -128,6 +128,7 @@ select = [
|
|||||||
"EM", # flake8-errmsg
|
"EM", # flake8-errmsg
|
||||||
"ICN", # flake8-import-conventions
|
"ICN", # flake8-import-conventions
|
||||||
"ISC", # flake8-implicit-str-concat
|
"ISC", # flake8-implicit-str-concat
|
||||||
|
"G", # flake8-logging-format
|
||||||
"PGH", # pygrep-hooks
|
"PGH", # pygrep-hooks
|
||||||
"PIE", # flake8-pie
|
"PIE", # flake8-pie
|
||||||
"PL", # pylint
|
"PL", # pylint
|
||||||
|
|||||||
Reference in New Issue
Block a user