🥅 Raise exception instead of warn + exit(1)

FCP-INDI · Aug 15, 2024 · 64a775b · 64a775b
1 parent 494f0c7
commit 64a775b
Showing 1 changed file with 3 additions and 4 deletions.
diff --git a/CPAC/_entrypoints/run.py b/CPAC/_entrypoints/run.py
@@ -786,10 +786,9 @@ def run_main():
                 # fillna
                 bids_table["ses"] = bids_table["ses"].fillna("None")
                 grouped_tab = bids_table.groupby(["sub", "ses"])
-            except Exception as e:  # TODO: raise exception
-                WFLOGGER.warning("Could not create bids table: %s", e)
-                print("Could not create bids table: %s", e)
-                sys.exit(1)
+            except Exception as e:
+                msg = f"Could not create bids table: {e}"
+                raise ValueError(msg) from e
         # else:
         #     sub_list = load_cpac_data_config(
         #         args.data_config_file, args.participant_label, args.aws_input_creds