chiark / gitweb /
lint: complain about UCMs that haven't been used
authorDaniel Martí <mvdan@mvdan.cc>
Wed, 5 Aug 2015 17:52:53 +0000 (10:52 -0700)
committerDaniel Martí <mvdan@mvdan.cc>
Wed, 5 Aug 2015 17:52:53 +0000 (10:52 -0700)
Mainly for MRs from people who don't fill the automatic fields

fdroidserver/lint.py

index 314366e1108121a863721bc740d1f86e88dc8205..678152dadfd9d99891d254564b9c480b92c11df9 100644 (file)
@@ -138,6 +138,8 @@ def main():
     allapps = metadata.read_metadata(xref=False)
     apps = common.read_app_args(args, allapps, False)
 
+    filling_ucms = re.compile('^(Tags.*|RepoManifest.*)')
+
     for appid, app in apps.iteritems():
         if app['Disabled']:
             continue
@@ -171,6 +173,14 @@ def main():
             if app['Web Site'].lower() == app['Source Code'].lower():
                 warn("Website '%s' is just the app's source code link" % app['Web Site'])
 
+        if filling_ucms.match(app['Update Check Mode']):
+            if all(app[f] == metadata.app_defaults[f] for f in [
+                    'Auto Name',
+                    'Current Version',
+                    'Current Version Code',
+                    ]):
+                warn("UCM is set but it looks like checkupdates hasn't been run yet")
+
         # Missing or incorrect categories
         if not app['Categories']:
             warn("Categories are not set")