From e85d1755f058e661e451f0ab77401f820d0c836e Mon Sep 17 00:00:00 2001 From: rubenwardy Date: Tue, 18 Aug 2020 18:10:42 +0100 Subject: Increase thread/comment ratelimiting based on rank --- app/tasks/minetestcheck/tree.py | 21 ++++++++++++--------- 1 file changed, 12 insertions(+), 9 deletions(-) (limited to 'app/tasks') diff --git a/app/tasks/minetestcheck/tree.py b/app/tasks/minetestcheck/tree.py index 07e67f6..57a769f 100644 --- a/app/tasks/minetestcheck/tree.py +++ b/app/tasks/minetestcheck/tree.py @@ -128,17 +128,20 @@ class PackageTreeNode: result["optional_depends"] = [] - # Check dependencies - for dep in result["depends"]: - if not basenamePattern.match(dep): - raise MinetestCheckError(("Invalid dependency name '{}' for mod at {}, names must only contain a-z0-9_.") \ - .format(dep, self.relative)) + def checkDependencies(deps): + for dep in result["depends"]: + if not basenamePattern.match(dep): + if " " in dep: + raise MinetestCheckError(("Invalid dependency name '{}' for mod at {}, did you forget a comma?") \ + .format(dep, self.relative)) + else: + raise MinetestCheckError(("Invalid dependency name '{}' for mod at {}, names must only contain a-z0-9_.") \ + .format(dep, self.relative)) - for dep in result["optional_depends"]: - if not basenamePattern.match(dep): - raise MinetestCheckError(("Invalid dependency name '{}' for mod at {}, names must only contain a-z0-9_.") \ - .format(dep, self.relative)) + # Check dependencies + checkDependencies(result["depends"]) + checkDependencies(result["optional_depends"]) # Fix games using "name" as "title" if self.type == ContentType.GAME: -- cgit v1.2.3