Skip to content

Commit

Permalink
Apply suggestions from code review
Browse files Browse the repository at this point in the history
Co-authored-by: Ronnie Dutta <61982285+MetRonnie@users.noreply.github.com>
  • Loading branch information
wxtim and MetRonnie committed Jul 17, 2024
1 parent 761a1fb commit 3006a97
Showing 1 changed file with 2 additions and 3 deletions.
5 changes: 2 additions & 3 deletions cylc/flow/scripts/validate_reinstall.py
Original file line number Diff line number Diff line change
Expand Up @@ -173,9 +173,8 @@ async def vr_cli(parser: COP, options: 'Values', workflow_id: str):
log_subcommand('validate --against-source', workflow_id)
await cylc_validate(parser, options, workflow_id)

# Turn against source options off again.
# Unset options that do not apply after validation:
delattr(options, 'against_source')
# Unset is validate after validation.
delattr(options, 'is_validate')

log_subcommand('reinstall', workflow_id)
Expand Down Expand Up @@ -203,7 +202,7 @@ async def vr_cli(parser: COP, options: 'Values', workflow_id: str):
'play',
unparsed_wid,
options,
compound_script_opts=VR_OPTIONS + [VALIDATE_AGAINST_SOURCE_OPTION],
compound_script_opts=[*VR_OPTIONS, VALIDATE_AGAINST_SOURCE_OPTION],
script_opts=(*PLAY_OPTIONS, *parser.get_std_options()),
source='', # Intentionally blank
)
Expand Down

0 comments on commit 3006a97

Please sign in to comment.