cli: remove auto-yes

materialsproject · Jun 26, 2023 · 74e085c · 74e085c
1 parent 063423f
commit 74e085c
Show file tree

Hide file tree

Showing 3 changed files with 1 addition and 9 deletions.
diff --git a/emmet-cli/emmet/cli/decorators.py b/emmet-cli/emmet/cli/decorators.py
@@ -127,10 +127,7 @@ def wrapper(*args, **kwargs):
             logger.debug(f"{track_dir} created")
 
         bb = ctx.grand_parent.params["bb"]
-        yes = ctx.grand_parent.params["yes"]
         if bb:
-            if not yes:
-                click.confirm("Did you run `module unload esslurm`?", abort=True)
             subdir = directory.rsplit(os.sep, 1)[1]
             stage_in = f"#DW stage_in source={directory} "
             stage_in += f"destination=$DW_JOB_STRIPED/{subdir} type=directory"
@@ -150,8 +147,6 @@ def wrapper(*args, **kwargs):
                 "time": "48:00:00",
             }
         else:
-            if not yes:
-                click.confirm("Did you run `module load esslurm`?", abort=True)
             slurm_kwargs = {
                 "qos": "xfer",
                 "time": "48:00:00",

diff --git a/emmet-cli/emmet/cli/entry_point.py b/emmet-cli/emmet/cli/entry_point.py
@@ -35,11 +35,10 @@ def opt_prompt():
     help="Number of jobs (for walltime > 48h).",
 )
 @click.option("--bb", is_flag=True, help="Use burst buffer.")
-@click.option("--yes", is_flag=True, help="Automatic yes to all prompts.")
 @click.option("--no-dupe-check", is_flag=True, help="Skip duplicate check(s).")
 @click.option("--verbose", is_flag=True, help="Show debug messages.")
 @click.version_option()
-def emmet(spec_or_dbfile, run, issue, sbatch, ntries, bb, yes, no_dupe_check, verbose):
+def emmet(spec_or_dbfile, run, issue, sbatch, ntries, bb, no_dupe_check, verbose):
     """Command line interface for emmet"""
     logger.setLevel(logging.DEBUG if verbose else logging.INFO)
     logging.getLogger("github3").setLevel(logging.WARNING)

diff --git a/emmet-cli/emmet/cli/tasks.py b/emmet-cli/emmet/cli/tasks.py
@@ -111,7 +111,6 @@ def backups(ctx, clean):
     rootdir_pattern = os.path.join(rootdir, ctx.parent.params["pattern"])
     comment_txt = [f"Backup {gen.value} launchers in `{rootdir_pattern}`:\n"]
     ctx.parent.parent.params["sbatch"] = True
-    ctx.parent.parent.params["yes"] = True
     run = ctx.parent.parent.params["run"]
     prefix = ctx.parent.params["pattern"] = "block_*"
 
@@ -505,7 +504,6 @@ def parsers(ctx, task_ids):
         remaining = group_strings_by_prefix(remaining_vaspdirs, len_prefix)
 
     ctx.parent.parent.params["sbatch"] = True
-    ctx.parent.parent.params["yes"] = True
     rootdir_pattern = os.path.join(directory, ctx.parent.params["pattern"])
     comment_txt = [f"Parse {gen.value} launchers in `{rootdir_pattern}`:\n"]