Skip to content

Commit

Permalink
Merge pull request #594 from linsword13/vet-dry-run
Browse files Browse the repository at this point in the history
Minor update on dry-run help texts
  • Loading branch information
douglasjacobsen authored Aug 5, 2024
2 parents 31bd906 + e2c1aa4 commit b0a44c9
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 4 deletions.
2 changes: 1 addition & 1 deletion lib/ramble/ramble/application.py
Original file line number Diff line number Diff line change
Expand Up @@ -1574,7 +1574,7 @@ def _analyze_experiments(self, workspace, app_inst=None):
"""

if self.get_status() == experiment_status.UNKNOWN.name and not workspace.dry_run:
logger.warn(f"Experiment has status is {self.get_status()}. Skipping analysis..\n")
logger.warn(f"Experiment has status {self.get_status()}. Skipping analysis..\n")
return

def format_context(context_match, context_format):
Expand Down
7 changes: 4 additions & 3 deletions lib/ramble/ramble/cmd/workspace.py
Original file line number Diff line number Diff line change
Expand Up @@ -511,7 +511,8 @@ def workspace_analyze_setup_parser(subparser):
"--dry-run",
dest="dry_run",
action="store_true",
help="perform a dry run. Allows progress on workspaces which are not fully setup",
help="perform a dry run. Allows going through analysis phases"
+ "on workspaces which are not fully setup",
)

subparser.add_argument(
Expand Down Expand Up @@ -945,8 +946,8 @@ def workspace_mirror_setup_parser(subparser):
dest="dry_run",
action="store_true",
help="perform a dry run. Creates package environments, "
+ "prints commands that would be executed "
+ "for installation, and files that would be downloaded.",
+ "prints package manager specific commands that would be executed "
+ "for creating the mirror.",
)

arguments.add_common_arguments(
Expand Down

0 comments on commit b0a44c9

Please sign in to comment.