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

Add current stashes to response. #12

Merged
merged 1 commit into from
Sep 7, 2015
Merged

Add current stashes to response. #12

merged 1 commit into from
Sep 7, 2015

Conversation

cmarcusreid
Copy link
Owner

Address #11. Need to verify that this does not substantially impact performance before merging. (If there is overhead, we can address by making the status query a separate request.)

@DoCode
Copy link

DoCode commented Sep 6, 2015

Many thanks...
I'll testing this in evening and give you some feedback

Sent from my Windows Phone


From: Marcus Reidmailto:notifications@github.com
Sent: ‎06.‎09.‎2015 16:47
To: cmarcusreid/git-status-cachemailto:git-status-cache@noreply.github.com
Subject: [git-status-cache] Add current stashes to response. (#12)

Address #11#11. Need to verify that this does not substantially impact performance before merging. (If there is overhead, we can address by making the status query a separate request.)


You can view, comment on, or merge this pull request online at:

#12

Commit Summary

  • Add current stashes to response.

File Changes

Patch Links:


Reply to this email directly or view it on GitHubhttps://github.com//pull/12.

@cmarcusreid
Copy link
Owner Author

I'll need to update the posh-git pull request before you will see this in the prompt, but you can see the raw information using Get-GitStatusFromCache.

@DoCode
Copy link

DoCode commented Sep 6, 2015

Working very fast on large repo (compiled with vs2015) :-)

@cmarcusreid
Copy link
Owner Author

I also measured on my end with 10 stashes and observed no performance impact. Looks like we're good to go!

cmarcusreid added a commit that referenced this pull request Sep 7, 2015
Add current stashes to response.
@cmarcusreid cmarcusreid merged commit aca7415 into master Sep 7, 2015
@cmarcusreid cmarcusreid deleted the supportStashes branch September 7, 2015 00:50
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