From 08ec61e96209a405a2b25a21d6d75a4532264ec1 Mon Sep 17 00:00:00 2001 From: Timothy Willard <9395586+TimothyWillard@users.noreply.github.com> Date: Tue, 12 Nov 2024 16:02:21 -0500 Subject: [PATCH] Fix bug in `--skip-manifest` flag The `--skip-manifest` flag had the opposite behaivor of what the name and help would suggest. --- flepimop/gempyor_pkg/src/gempyor/batch.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/flepimop/gempyor_pkg/src/gempyor/batch.py b/flepimop/gempyor_pkg/src/gempyor/batch.py index 1dd2d9531..2510dfc33 100644 --- a/flepimop/gempyor_pkg/src/gempyor/batch.py +++ b/flepimop/gempyor_pkg/src/gempyor/batch.py @@ -1084,6 +1084,8 @@ def _click_submit(ctx: click.Context = mock_context, **kwargs: Any) -> None: # Manifest if kwargs["skip_manifest"]: + logger.debug("Skipped writing manifest metadata.") + else: manifest = write_manifest( job_name, kwargs["flepi_path"], @@ -1091,8 +1093,6 @@ def _click_submit(ctx: click.Context = mock_context, **kwargs: Any) -> None: destination=kwargs["project_path"] / "manifest.json", ) logger.info("Writing manifest metadata to '%s'", manifest.absolute()) - else: - logger.debug("Skipped writing manifest metadata.") # Config out if (config_out := kwargs["config_out"]) is None: