Paths

Table of Contentst

Differential D8300

Add force flag to skip prompt when deleting runs from cli
ClosedPublic
Actions

Authored by • rexledesma on Jun 9 2021, 12:45 AM.

Details

Reviewers

alangenfeld

Commits

R1:60b7b7544f76: Add force flag to skip prompt when deleting runs from cli

Summary

Resolves https://github.com/dagster-io/dagster/issues/4268.

Also, make the unhappy paths raise a ClickException to return a non-zero exit code.

Test Plan

pytest

Diff Detail

Repository

R1 dagster

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

• rexledesma created this revision.Jun 9 2021, 12:45 AM

• rexledesma edited the summary of this revision. (Show Details)Jun 9 2021, 12:46 AM

Harbormaster completed remote builds in B31858: Diff 39237.Jun 9 2021, 1:10 AM

• rexledesma requested review of this revision.Jun 9 2021, 1:10 AM

graveler

This revision is now accepted and ready to land.Jun 9 2021, 3:06 PM

Closed by commit R1:60b7b7544f76: Add force flag to skip prompt when deleting runs from cli (authored by • rexledesma). · Explain WhyJun 9 2021, 3:13 PM

This revision was automatically updated to reflect the committed changes.

• rexledesma added a commit: R1:60b7b7544f76: Add force flag to skip prompt when deleting runs from cli.

Revision Contents
Changeset List

Path

Size

python_modules/

dagster/

cli/

run.py

48 lines

dagster_tests/

cli_tests/

command_tests/

test_cli_commands.py

25 lines

Diff 39264

View Options

python_modules/dagster/dagster/cli/run.py