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

some SSL certificate bundle tweaks #116

Merged
merged 4 commits into from
Sep 7, 2017
Merged

Conversation

shiftkey
Copy link
Member

@shiftkey shiftkey commented Sep 5, 2017

Fixes #87
Fixes #86

  • this will probably break the linux tests

cc @kittaakos

@kittaakos
Copy link
Contributor

Awesome! I have just made some changes to be able to run tests with external Git too. Just in case if you are interested, I can make some contributions: master...kittaakos:GH-87

@shiftkey
Copy link
Member Author

shiftkey commented Sep 5, 2017

@kittaakos oh that's cool - I'd love to get those external tests in too if you're interested in building upon this work!

@@ -1,5 +1,11 @@
import * as path from 'path'

function resolveEmbeddedGitDir(): string {
const s = path.sep;

This comment was marked as spam.

This comment was marked as spam.

@shiftkey shiftkey merged commit 0869597 into master Sep 7, 2017
@shiftkey shiftkey deleted the support-ssl-environments branch September 7, 2017 07:28
@shiftkey shiftkey mentioned this pull request Sep 7, 2017
2 tasks
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