Skip to content

Better support for merging when in dirty state#1416

Merged
RMacfarlane merged 2 commits intomasterfrom
jackson/pr-uncommitted-changes
Oct 28, 2019
Merged

Better support for merging when in dirty state#1416
RMacfarlane merged 2 commits intomasterfrom
jackson/pr-uncommitted-changes

Conversation

@JacksonKearl
Copy link
Contributor

@JacksonKearl JacksonKearl commented Oct 25, 2019

Fixes #1405

Adds:

  • When trying to merge with a dirty directory or unpushed changes, user gets a confirmation dialog
  • When trying to delete a branch post-merge, if the branch has uncommitted changes, the user gets a confirmation dialog to discard them.

@JacksonKearl JacksonKearl self-assigned this Oct 26, 2019

await this._pullRequestManager.repository.deleteBranch(branchInfo!.branch, true);
resolve();
});
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure why this was done with explicit promise construction?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

a mystery 🤷‍♀

Copy link
Contributor

@RMacfarlane RMacfarlane left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@RMacfarlane RMacfarlane merged commit 27fc484 into master Oct 28, 2019
@RMacfarlane RMacfarlane deleted the jackson/pr-uncommitted-changes branch October 28, 2019 17:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Handle checkout failure when deleting pull request

2 participants