Skip to content

Commit

Permalink
Merge pull request buildbot#7940 from pmisik/dep-update-2024-w33-ruff
Browse files Browse the repository at this point in the history
Update ruff from 0.5.2 to 0.6.1
  • Loading branch information
p12tic authored Aug 21, 2024
2 parents 83dd162 + 3e77049 commit 4d1c163
Show file tree
Hide file tree
Showing 6 changed files with 8 additions and 2 deletions.
2 changes: 1 addition & 1 deletion Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -95,7 +95,7 @@ rmpyc:
find master worker \( -name '*.pyc' -o -name '*.pyo' \) -exec rm -v {} \;

ruff:
ruff --fix .
ruff format .

docker: docker-buildbot-worker docker-buildbot-master
echo done
Expand Down
3 changes: 3 additions & 0 deletions master/buildbot/test/unit/util/test_interfaces.py
Original file line number Diff line number Diff line change
Expand Up @@ -32,6 +32,7 @@ def myfunc2(x, y=2, *args):
@self.assertArgSpecMatches(myfunc)
def myfunc3(x, y=3, *args):
pass

except Exception as e:
error = e
else:
Expand Down Expand Up @@ -59,6 +60,7 @@ def myfunc4(x, y):
@self.assertArgSpecMatches(myfunc, myfunc3)
def myfunc5(x, y):
pass

except Exception as e:
error = e
else:
Expand All @@ -72,6 +74,7 @@ def myfunc5(x, y):
@self.assertArgSpecMatches(myfunc, myfunc3)
def myfunc6(xx, yy):
pass

except Exception as e:
error = e
else:
Expand Down
1 change: 1 addition & 0 deletions master/buildbot/test/util/endpoint.py
Original file line number Diff line number Diff line change
Expand Up @@ -106,6 +106,7 @@ def test_get_spec(self):
@self.assertArgSpecMatches(self.ep.get)
def get(self, resultSpec, kwargs):
pass

except trial.unittest.FailTest:

@self.assertArgSpecMatches(self.ep.get)
Expand Down
2 changes: 1 addition & 1 deletion requirements-ci.txt
Original file line number Diff line number Diff line change
Expand Up @@ -62,7 +62,7 @@ pytz==2024.1
PyYAML==6.0.1
requests==2.32.3
responses==0.25.3
ruff==0.5.2
ruff==0.6.1
s3transfer==0.10.2
service-identity==24.1.0
setuptools-trial==0.6.0
Expand Down
1 change: 1 addition & 0 deletions worker/buildbot_worker/commands/fs.py
Original file line number Diff line number Diff line change
Expand Up @@ -195,6 +195,7 @@ def eb(f):
@d.addCallback
def send_rc(rc):
self.sendStatus([('rc', rc)])

else:
if not os.path.exists(os.path.dirname(to_path)):
os.makedirs(os.path.dirname(to_path))
Expand Down
1 change: 1 addition & 0 deletions worker/buildbot_worker/commands/utils.py
Original file line number Diff line number Diff line change
Expand Up @@ -103,6 +103,7 @@ def rmdirRecursive(dir):
os.chmod(full_name, 0o700)
os.remove(full_name)
os.rmdir(dir)

else:
# use rmtree on POSIX
import shutil
Expand Down

0 comments on commit 4d1c163

Please sign in to comment.