Skip to content

Conversation

@axosoft-ramint
Copy link
Contributor

Closes #4124

Note: there are known issues with the "delete worktree" case where when the worktree is deleted before deleting the branch, the "delete branch" portion of the commands fail to proceed because the commands are using a worktree path that no longer exists. These should be addressed in follow-up.

@axosoft-ramint axosoft-ramint requested review from d13 and eamodio March 4, 2025 18:33
@axosoft-ramint axosoft-ramint force-pushed the feature/merged-actions branch 3 times, most recently from 865cada to 3e14005 Compare March 10, 2025 16:51
Copy link
Member

@eamodio eamodio left a comment

Choose a reason for hiding this comment

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

Looks good! Approved with minor tweaks

});
}

private async deleteBranchOrWorktree(ref: BranchRef, mergeTarget?: BranchRef) {
Copy link
Member

Choose a reason for hiding this comment

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

Is there a confirmation for the delete happening or just the "Proceed" warnings below? If there is no other confirmations, then I think the wording need to be changed from what is below or suggested.

Copy link
Contributor Author

@axosoft-ramint axosoft-ramint Mar 11, 2025

Choose a reason for hiding this comment

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

I believe it just starts the "delete branch" flow, which includes quickpick steps/confirmations in it. It would be the same as choosing "delete branch" from a context menu, the only difference being the branch/worktree switch preceding it.

@axosoft-ramint axosoft-ramint force-pushed the feature/merged-actions branch from 3e14005 to a2da46c Compare March 11, 2025 17:21
@axosoft-ramint axosoft-ramint merged commit 7efdd2f into main Mar 11, 2025
3 checks passed
@axosoft-ramint axosoft-ramint deleted the feature/merged-actions branch March 11, 2025 17:52
saeedzaha pushed a commit to saeedzaha/vscode-gitlens that referenced this pull request Apr 28, 2025
* Adds merge detection actions (push and delete) & squash merge detection on the merge target section in Home view.
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.

Add squash merge detection and merged target actions

3 participants