Mirror
/
zulip
mirror of https://github.com/zulip/zulip.git
1
0
Fork 0
Code Issues Projects Releases Wiki Activity

python: Use ‘not in’ for more negated membership tests. 

Fixes “E713 Test for membership should be `not in`” found by ruff (now
that I’ve fixed it not to ignore scripts lacking a .py extension).

Signed-off-by: Anders Kaseorg <anders@zulip.com>
This commit is contained in:
Anders Kaseorg 2022-09-25 16:56:42 -07:00 committed by Anders Kaseorg
parent 4ba9853481
commit b267b17677
2 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
scripts/nagios/check-rabbitmq-consumers
View File

@ -19,7 +19,7 @@ states = {
3: "UNKNOWN", 3: "UNKNOWN",
} }
if "USER" in os.environ and not os.environ["USER"] in ["root", "rabbitmq"]: if "USER" in os.environ and os.environ["USER"] not in ["root", "rabbitmq"]:
print("This script must be run as the root or rabbitmq user") print("This script must be run as the root or rabbitmq user")

2
tools/test-js-with-node
View File

@ -399,7 +399,7 @@ def enforce_proper_coverage(coverage_json: Any) -> bool:
coverage_lost = False coverage_lost = False
for relative_path in enforce_fully_covered: for relative_path in enforce_fully_covered:
path = ROOT_DIR + "/" + relative_path path = ROOT_DIR + "/" + relative_path
if not (path in coverage_json): if path not in coverage_json:
coverage_lost = True coverage_lost = True
print_error(f"{relative_path} has no node test coverage") print_error(f"{relative_path} has no node test coverage")
continue continue