Fixed #524 - Remove irrelevant error for attrib

Signed-off-by: Chin Yeung Li <[email protected]>

Author: chinyeungli

CHANGELOG.rst

@@ -1,9 +1,10 @@
 ==============================
 Changelog
-2023-03-07
-    Release 7.2.1
+2023-xx-xx
+    Release 7.3.0
 
     * Fixed the transform code for xlsx and json
+    * Remove irrelevant error for attrib
 
 
 2022-10-24

src/attributecode/cmd.py

@@ -493,23 +493,25 @@ def attrib(input, output, api_url, api_key, scancode, min_license_score, referen
         if not reference:
             # Set current directory as the reference dir
             reference = os.path.dirname(input)
-        errors, abouts = load_inventory(
+        # Since the errors from load_inventory is only about field formatting or
+        # empty field which is irrelevant for attribtion process,
+        # See https://github.com/nexB/aboutcode-toolkit/issues/524
+        # I believe we do not need to capture these errors in attrib process
+        _errors, abouts = load_inventory(
             location=input,
             from_attrib=from_attrib,
             scancode=scancode,
             reference_dir=reference
         )
+
     else:
         is_about_input = True
-        errors, abouts = collect_inventory(input)
+        _errors, abouts = collect_inventory(input)
 
     if not abouts:
-        if errors:
-            errors_count = report_errors(errors, quiet, verbose, log_file_loc=output + '-error.log')
-        else:
-            msg = 'No ABOUT file or reference is found from the input. Attribution generation halted.'
-            click.echo(msg)
-            errors_count = 1
+        msg = 'No ABOUT file or reference is found from the input. Attribution generation halted.'
+        click.echo(msg)
+        errors_count = 1
         sys.exit(errors_count)
 
     if not is_about_input: