This is an automated email from the ASF dual-hosted git repository.
sbp pushed a commit to branch sbp
in repository https://gitbox.apache.org/repos/asf/tooling-trusted-releases.git
The following commit(s) were added to refs/heads/sbp by this push:
new 1e83287 Fix unparenthesized subexpressions
1e83287 is described below
commit 1e83287cd6756d8dae6ab7714c4b734b0e1cfe0a
Author: Sean B. Palmer <[email protected]>
AuthorDate: Fri Jan 30 20:34:47 2026 +0000
Fix unparenthesized subexpressions
---
atr/get/finish.py | 2 +-
atr/ssh.py | 4 ++--
atr/storage/writers/tokens.py | 2 +-
3 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/atr/get/finish.py b/atr/get/finish.py
index c687298..723a8e7 100644
--- a/atr/get/finish.py
+++ b/atr/get/finish.py
@@ -557,7 +557,7 @@ def _render_release_card(release: sql.Release,
announce_disable_message: str) ->
project_name=release.project.name,
version_name=release.version,
)
- if not announce_disable_message
+ if (not announce_disable_message)
else None,
)[
htm.icon("check-circle"),
diff --git a/atr/ssh.py b/atr/ssh.py
index 2a5b9ad..1f5f287 100644
--- a/atr/ssh.py
+++ b/atr/ssh.py
@@ -357,7 +357,7 @@ def _step_05a_command_path_validate_read(path: str) ->
tuple[str, str, str | Non
if "//" in path:
raise RsyncArgsError("The path argument should not contain //")
- if path.count("/") < 3 or path.count("/") > 4:
+ if (path.count("/") < 3) or (path.count("/") > 4):
raise RsyncArgsError("The path argument should be a
/PROJECT/VERSION/(tag)/ directory path")
path_project, path_version, *rest = path.strip("/").split("/", 2)
@@ -452,7 +452,7 @@ async def _step_06a_validate_read_permissions(
)
if tag:
- if not release.project.release_policy or (not
release.project.release_policy.file_tag_mappings):
+ if (not release.project.release_policy) or (not
release.project.release_policy.file_tag_mappings):
raise RsyncArgsError(f"Release '{release.name}' does not support
tags")
tags = release.project.release_policy.file_tag_mappings.keys()
if tag not in tags:
diff --git a/atr/storage/writers/tokens.py b/atr/storage/writers/tokens.py
index 1c875f5..d47cefa 100644
--- a/atr/storage/writers/tokens.py
+++ b/atr/storage/writers/tokens.py
@@ -120,7 +120,7 @@ class FoundationCommitter(GeneralPublic):
# Verify account still exists in LDAP
account_details = await ldap.account_lookup(self.__asf_uid)
- if account_details is None or ldap.is_banned(account_details):
+ if (account_details is None) or ldap.is_banned(account_details):
raise storage.AccessError("Authentication failed")
issued_jwt = jwtoken.issue(self.__asf_uid)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
