Merge pull request #177 from jepler/warn-no-descr-repos

Print a warning for repos without descriptions

Author: sommersoft

adabot/circuitpython_libraries.py

@@ -231,13 +231,14 @@ def run_library_checks(validators, bundle_submodules, latest_pylint, kw_args):
         int(pr_sort_re.search(pr).group(1)) for pr in lib_insights["open_prs"]
         if pr_sort_re.search(pr) is not None
     ]
-    output_handler(
-        "  * {0} open pull requests (Oldest: {1}, Newest: {2})".format(
-            len(lib_insights["open_prs"]),
-            max(open_pr_days),
-            max((min(open_pr_days), 1)) # ensure the minumum is '1'
+    if len(lib_insights["open_prs"]) != 0:
+        output_handler(
+            "  * {0} open pull requests (Oldest: {1}, Newest: {2})".format(
+                len(lib_insights["open_prs"]),
+                max(open_pr_days),
+                max((min(open_pr_days), 1)) # ensure the minumum is '1'
+            )
         )
-    )
     output_handler("Library updates in the last seven days:")
     if len(new_libs) != 0:
         output_handler("**New Libraries**")

adabot/lib/circuitpython_library_validators.py

@@ -56,6 +56,7 @@
 " without reference to import errno.  See issue " \
 "https://github.com/adafruit/circuitpython/issues/1582"
 ERROR_MISMATCHED_READTHEDOCS = "Mismatched readthedocs.yml"
+ERROR_MISSING_DESCRIPTION = "Missing repository description"
 ERROR_MISSING_EXAMPLE_FILES = "Missing .py files in examples folder"
 ERROR_MISSING_EXAMPLE_FOLDER = "Missing examples folder"
 ERROR_EXAMPLE_MISSING_SENSORNAME = "Example file(s) missing sensor/library name"
@@ -209,6 +210,9 @@ def validate_repo_state(self, repo):
 
         errors = []
 
+        if not repo_fields.get("description"):
+            errors.append(ERROR_MISSING_DESCRIPTION)
+
         if repo_fields.get("has_wiki"):
             errors.append(ERROR_WIKI_DISABLED)