Don't emit an additional error in the asset handler when a previous error occurs

Author: djungelorm

CHANGES.txt

@@ -1,3 +1,6 @@
+v1.1.6
+ * Don't emit an additional error in the copy assets handler if the build process fails
+
 v1.1.5
  * Add sphinx_tabs_nowarn option to disable warning about incompatible builder. Useful when running sphinx-build with the -W flag
 

sphinx_tabs/tabs.py

@@ -256,16 +256,14 @@ def copy_assets(app, exception):
                 'readthedocs', 'readthedocsdirhtml',
                 'readthedocssinglehtml', 'readthedocssinglehtmllocalmedia']
     builders.extend(app.config['sphinx_tabs_valid_builders'])
+    if exception:
+        return
     if app.builder.name not in builders:
         if not app.config['sphinx_tabs_nowarn']:
             app.warn(
                 'Not copying tabs assets! Not compatible with %s builder' %
                 app.builder.name)
         return
-    if exception:
-        if not app.config['sphinx_tabs_nowarn']:
-            app.warn('Not copying tabs assets! Error occurred previously')
-        return
     app.info('Copying tabs assets... ', nonl=True)
 
     installdir = os.path.join(app.builder.outdir, '_static', 'sphinx_tabs')