Skip to content

Commit c48efb2

Browse files
authored
Merge pull request #72 from Mic92/fixes
fixup logger warning call
2 parents 4bb5e60 + 6882c81 commit c48efb2

File tree

2 files changed

+6
-2
lines changed

2 files changed

+6
-2
lines changed

buildbot_nix/__init__.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -684,15 +684,15 @@ def check_builder(
684684

685685
builder = yield self.master.data.get(("builders", res["builderid"]))
686686
if builder["name"] in self.builders:
687-
log.warning(
687+
log.warn(
688688
"Builder {builder} allowed by {role}: {builders}",
689689
builder=builder["name"],
690690
role=self.role,
691691
builders=self.builders,
692692
)
693693
return Match(self.master, **{object_type: res})
694694
else:
695-
log.warning(
695+
log.warn(
696696
"Builder {builder} not allowed by {role}: {builders}",
697697
builder=builder["name"],
698698
role=self.role,

pyproject.toml

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -79,6 +79,10 @@ ignore = [
7979
"E501", # line too long
8080
"T201", # `print` found
8181
"PLR2004", # Magic value used in comparison
82+
83+
# not compatible with twisted logger: https://docs.twisted.org/en/twisted-18.7.0/core/howto/logger.html
84+
"PGH002", # `warn` is deprecated in favor of `warning`
85+
"G010", # Logging statement uses `warn` instead of `warning`
8286
]
8387

8488
[tool.mypy]

0 commit comments

Comments
 (0)