Merge pull request #1063 from googlefonts/qa-run

qa: carry on runnning all tools if an error occurs

Lib/gftools/qa.py

@@ -42,6 +42,7 @@ def __init__(self, fonts, fonts_before=None, out="out", url=None):
         self.fonts_before = fonts_before
         self.out = out
         self.url = url
+        self.has_error = False
 
     @report_exceptions
     def diffenator(self, **kwargs):
@@ -134,8 +135,7 @@ def fontbakery(self, profile="googlefonts", html=False, extra_args=None):
             self.post_to_github(msg)
 
         if process.returncode != 0:
-            logger.fatal("Fontbakery has raised a fatal error. Please fix!")
-            sys.exit(1)
+            self.has_error = True
 
     def googlefonts_upgrade(self, imgs=False):
         self.fontbakery()

Lib/gftools/scripts/qa.py

@@ -20,6 +20,7 @@
 from fontTools.ttLib import TTFont
 import argparse
 import os
+import sys
 import shutil
 import logging
 from gftools.utils import (
@@ -268,6 +269,10 @@ def main(args=None):
     if args.interpolations:
         qa.interpolations()
 
+    if qa.has_error:
+        logger.fatal("Fontbakery has raised a fatal error. Please fix!")
+        sys.exit(1)
+
 
 if __name__ == "__main__":
     main()