Skip to content

Added SSL support#10

Closed
d-chugunov wants to merge 2 commits intoClever:masterfrom
d-chugunov:ssl
Closed

Added SSL support#10
d-chugunov wants to merge 2 commits intoClever:masterfrom
d-chugunov:ssl

Conversation

@d-chugunov
Copy link

No description provided.

README.md Outdated
Copy link
Contributor

Choose a reason for hiding this comment

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

Can you make this a boolean parameter instead of defined vs undefined, and then it would default false?

@Sammons
Copy link

Sammons commented Feb 14, 2016

+1, really would like this

@azylman
Copy link
Contributor

azylman commented Feb 16, 2016

Hey @d-chugunov,

Thanks for making those updates! This isn't mergeable right now, if you rebase it against latest master and ping me I'll merge it ASAP.

Thanks so much!

@d-chugunov
Copy link
Author

@azylman ,
I did my best in rebasing. So you may try to merge my master branch with yours.

@azylman
Copy link
Contributor

azylman commented Feb 16, 2016

@d-chugunov Did you push your new commit after rebasing? This still says it can't be merged

@d-chugunov
Copy link
Author

@azylman Yes, I pushed them. Try to merge from pull request #9

@azylman azylman mentioned this pull request Feb 16, 2016
@azylman azylman closed this in 91bd170 Feb 16, 2016
@azylman
Copy link
Contributor

azylman commented Feb 16, 2016

This is published in 1.3.0

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.

3 participants

Comments