Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Replace git command with git extension api (1/2) #410

Merged
merged 24 commits into from
Feb 8, 2020
Merged

Replace git command with git extension api (1/2) #410

merged 24 commits into from
Feb 8, 2020

Conversation

ole1986
Copy link
Contributor

@ole1986 ole1986 commented Nov 26, 2019

This PR targets on replacing git commands with the native git extension API (v1).
The following parts will be replaced with this PR

  • Amended GitServiceFactory and GitService to use git extension API
  • Initialize GitService with its current repository (or selected as default)
  • getBranches
  • getHeadHashes
  • getGitRoot
  • getCurrentBranch new
  • getOriginUrl
  • getRefsContainingCommit

Also, some methods related to git may have be removed (like getHash) as they are no longer in needed

PLEASE NOTE: Behavior has changed for repositories

In previous versions a prompt may appear for existing repositories (E.g. submodules) from inside the workspace. In future this is also handled by the Git Extension API which may require to select the repository from source code provider side bar before running the command Git History (git log)

@ole1986
Copy link
Contributor Author

ole1986 commented Dec 31, 2019

@DonJayamanne would you mind checking this PR? Especially the use of submodules (or multiple git repositories in same workspace) as the behavior has slightly changed

@ole1986 ole1986 force-pushed the git-api branch 2 times, most recently from ff65444 to e41f951 Compare January 9, 2020 08:46
@ole1986 ole1986 force-pushed the git-api branch 4 times, most recently from d33777c to a9f9fbf Compare February 5, 2020 09:04
@ole1986 ole1986 changed the title Replace git command with git extension api Replace git command with git extension api (1/2) Feb 7, 2020
@ole1986
Copy link
Contributor Author

ole1986 commented Feb 7, 2020

I think I am going to split this into two parts since vscode related issues are not yet included in the last insider version

@ole1986 ole1986 merged commit e8becc4 into master Feb 8, 2020
@ole1986 ole1986 deleted the git-api branch February 8, 2020 13:11
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