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

Use vscode.workspace.workspaceFolders[0] to construct SimpleGit instance. #268

Merged
merged 1 commit into from
Jan 16, 2022

Conversation

lukester1975
Copy link
Contributor

Feature

Or is it a fix...

Allows a per-workspace git user and email address.

Note that this change doesn't cope with multiple workspaces in a single vscode instance. It would probably have to defer the config lookup and use the appropriate workspace root for the file being edited at that point.

Thanks for the extension, hope this is useful and it should at least be good for most cases!

Regards

@lukester1975 lukester1975 changed the title Use vscode.workspace.rootPath to construct SimpleGit instance. Use vscode.workspace.workspaceFolders[0] to construct SimpleGit instance. Dec 29, 2021
Allows per-workspace git user and email address.
@codecov
Copy link

codecov bot commented Dec 30, 2021

Codecov Report

Merging #268 (e2aa1cb) into master (a5b5d32) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #268   +/-   ##
=======================================
  Coverage   93.36%   93.36%           
=======================================
  Files           5        5           
  Lines         814      814           
  Branches      242      242           
=======================================
  Hits          760      760           
  Misses         54       54           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a5b5d32...e2aa1cb. Read the comment docs.

@cschlosser cschlosser merged commit 9c8c583 into cschlosser:master Jan 16, 2022
@lukester1975 lukester1975 deleted the workspace-dir-fix branch January 16, 2022 20:22
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