chiark / gitweb /
Re-add https enforcings to Repo:
authorDaniel Martí <mvdan@mvdan.cc>
Wed, 3 Jun 2015 20:17:35 +0000 (22:17 +0200)
committerDaniel Martí <mvdan@mvdan.cc>
Wed, 3 Jun 2015 20:17:57 +0000 (22:17 +0200)
fdroidserver/lint.py

index d1d0c5226b84043b1f0c5746ed16c0fcd8fd9ca9..bd587cbed836da397893bbbc02e8f9d0844f678d 100644 (file)
@@ -32,15 +32,18 @@ def enforce_https(domain):
     return (re.compile(r'.*[^sS]://[^/]*' + re.escape(domain) + r'/.*'),
             domain + " URLs should always use https://")
 
-http_warnings = [
-    (re.compile(r'.*/$'),
-     "HTTP links shouldn't end with a slash"),
+https_enforcings = [
     enforce_https('github.com'),
     enforce_https('gitorious.org'),
     enforce_https('apache.org'),
     enforce_https('google.com'),
     enforce_https('svn.code.sf.net'),
     enforce_https('googlecode.com'),
+]
+
+http_warnings = https_enforcings + [
+    (re.compile(r'.*/$'),
+     "HTTP links shouldn't end with a slash"),
     # TODO enable in August 2015, when Google Code goes read-only
     # (re.compile(r'.*://code\.google\.com/.*'),
     #  "code.google.com will be soon switching down, perhaps the project moved to github.com?"),
@@ -51,7 +54,7 @@ regex_warnings = {
     ],
     'Source Code': http_warnings + [
     ],
-    'Repo': [
+    'Repo': https_enforcings + [
     ],
     'Issue Tracker': http_warnings + [
         (re.compile(r'.*github\.com/[^/]+/[^/]+[/]*$'),
@@ -96,10 +99,6 @@ regex_pedantic = {
          "Appending .git is not necessary"),
     ],
     'Repo': [
-        (re.compile(r'^http://.*'),
-         "use https:// if available"),
-        (re.compile(r'^svn://.*'),
-         "use https:// if available"),
     ],
     'Issue Tracker': [
         (re.compile(r'.*github\.com/[^/]+/[^/]+/issues/.*'),