Skip to content

Add commands for stash explorer#102

Closed
rebornix wants to merge 17 commits intogitkraken:feature-explorerfrom
rebornix:feature-explorer
Closed

Add commands for stash explorer#102
rebornix wants to merge 17 commits intogitkraken:feature-explorerfrom
rebornix:feature-explorer

Conversation

@rebornix
Copy link
Contributor

The idea is

  • Bind existing command to stash commit node
  • Check if the args is commit node in Stash command, if so, directly read stash commit info from it instead of fetching that again from local
  • Run command
  • Trigger the update of the tree

Right now I reference types from View, which is not good. @eamodio let me know your take on this.

}

async execute(args: StashDeleteCommandArgs = { confirm: true }) {
async execute(args: StashDeleteCommandArgs | StashCommitNode = { confirm: true }) {
Copy link
Member

Choose a reason for hiding this comment

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

@rebornix Can we use the new run method like here: https://github.com/eamodio/vscode-gitlens/blob/feature-explorer/src/commands/diffWithPrevious.ts#L25

That way the dealing with the node and refreshing it could be specific for that context?

@eamodio eamodio force-pushed the feature-explorer branch 6 times, most recently from c60eece to 2f8cfb1 Compare June 29, 2017 05:54
@eamodio
Copy link
Member

eamodio commented Jun 29, 2017

@rebornix sorry for the mess I made out of this PR with all the rebasing. I've "merged" this in to the feature-explorer branch, but also heavily modified it to fit with the new command structure. And also changed the view refreshing to be decoupled from the commands themselves -- since they could be executed in other contexts (or even update via command line changes).

Let me know what you think.

@eamodio eamodio force-pushed the feature-explorer branch from 2f8cfb1 to d288985 Compare July 3, 2017 02:56
@eamodio
Copy link
Member

eamodio commented Jul 20, 2017

This has been "merged" closing...

@eamodio eamodio closed this Jul 20, 2017
billsedison added a commit to noside911/vscode-gitlens that referenced this pull request Jan 4, 2019
Revert "Added documentation for packaging and fixed some issues for prod environment"
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.

2 participants