Skip to content
This repository has been archived by the owner on Sep 23, 2024. It is now read-only.

Commit

Permalink
Fix pylint W1201: logging-not-lazy
Browse files Browse the repository at this point in the history
Signed-off-by: Major Hayden <[email protected]>
  • Loading branch information
major committed May 22, 2018
1 parent 81dcb8d commit 1df028a
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 5 deletions.
8 changes: 5 additions & 3 deletions skt/__init__.py
Original file line number Diff line number Diff line change
Expand Up @@ -294,8 +294,10 @@ def getrname(self, uri):
if not uri.endswith('/')
else uri.split('/')[-2].replace('.git', ''))
while self.get_remote_url(rname) == uri:
logging.warning("remote '%s' already exists with a different uri, "
"adding '_'" % rname)
logging.warning(
"remote '%s' already exists with a different uri, adding '_'",
rname
)
rname += '_'

return rname
Expand Down Expand Up @@ -428,7 +430,7 @@ def bisect_iter(self, bad):
if m:
binfo = m.group(1)
ret = 1
logging.warning("Bisected, bad commit: %s" % binfo)
logging.warning("Bisected, bad commit: %s", binfo)
break
else:
logging.info(line)
Expand Down
4 changes: 2 additions & 2 deletions skt/runner.py
Original file line number Diff line number Diff line change
Expand Up @@ -395,10 +395,10 @@ def gethost(self, jobid=None):
if jobid is None:
jobid = self.lastsubmitted

logging.info("gethost for %s" % jobid)
logging.info("gethost for %s", jobid)
root = self.getresultstree(jobid)
recipe = root.find("recipeSet/recipe")
logging.info("%s: %s" % (jobid, recipe.attrib.get("system")))
logging.info("%s: %s", jobid, recipe.attrib.get("system"))

return recipe.attrib.get("system")

Expand Down

0 comments on commit 1df028a

Please sign in to comment.