Merging a branch should default to performing a dry run

[jeremystretch]

Plugin Version

v0.2.0

Proposed functionality

The "merge branch" confirmation dialog (pictured below) currently commits by default.

This FR proposes rendering this checkbox as unchecked initially.

Use case

Leaving this checkbox unchecked initially provides a safer default action (i.e. performing a dry run). It would also better abide by the convention of requiring an additional click to perform a potentially dangerous operation.

External dependencies

No response

[linear]

BRA-6 Merging a branch should default to performing a dry run

Plugin Version

v0.2.0

Proposed functionality

The "merge branch" confirmation dialog (pictured below) currently commits by default.

screenshot

This FR proposes rendering this checkbox as unchecked initially.

Use case

Leaving this checkbox unchecked initially provides a safer default action (i.e. performing a dry run). It would also better abide by the convention of requiring an additional click to perform a potentially dangerous operation.

External dependencies

No response

[mrmrcoleman]

"Commit" may have confusing connotations in this context. "Dry Run" with reversed logic would be clearer.