part-validate-parts: Better output, yielding a message even if

everything is ok.
ninja: Include the digikey.dl as a dependency even if custom is used.
digikey.dl: More rules.

1 files changed, 4 insertions, 2 deletions

diff --git a/src/ee/tools/part_validate_parts.py b/src/ee/tools/part_validate_parts.py
index a9c5ba7..65477e5 100644
--- a/src/ee/tools/part_validate_parts.py
+++ b/src/ee/tools/part_validate_parts.py
@@ -80,7 +80,7 @@ def check_has_footprint(ctx: Context, bom: Part, sch: Part):
         return ms.warning("Part has footprint in schematic and a BOM part but their footprints do not match. "
                           f"Footprints: part: {fp}, BOM: {supplier_fp}.")
 
-    ms.info("Part has footprint and BOM part. Their footprints matches.")
+    return ms.info("Part has footprint and BOM part. Their footprints matches.")
 
 
 def validate(f, ctx: Context, bom_part: Part, sch_part: Part):
@@ -144,7 +144,7 @@ def work(bom_path: Path, sch_path: Path, report_path: Path, part_dbs: List[Path]
         print("Message index", file=f)
         print("=============", file=f)
         print("", file=f)
-        for message, messages_ in groupby(messages.messages, lambda m: m[2]):
+        for message, messages_ in groupby(sorted(messages.messages, key=lambda m: m[2]), lambda m: m[2]):
             messages = list(messages_)
             print(message, file=f)
             print("{}".format("-" * len(message)), file=f)
@@ -152,6 +152,8 @@ def work(bom_path: Path, sch_path: Path, report_path: Path, part_dbs: List[Path]
             for m in messages:
                 print(" * {}".format(m[0].printable_reference), file=f)
 
+            print("", file=f)
+
 
 parser = argparse.ArgumentParser()
 ee.tools.add_default_argparse_group(parser)