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 https support #202

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

jonnyka
Copy link

@jonnyka jonnyka commented Jun 19, 2018

solution for #199

@rwky
Copy link

rwky commented Jun 19, 2018

Travis is failing but appears to be due to old node versions, perhaps anything less than 6 should be removed from travis since they're not supported upstream @dougwilson what do you think?

Apart from that @jonnyka can you add tests and update the README?

@jonnyka
Copy link
Author

jonnyka commented Jun 20, 2018

@rwky yes will do, I'm a bit swamped today, probably gonna update the PR tomorrow

@oscl
Copy link

oscl commented Jun 21, 2018

Awesome!

@jonnyka
Copy link
Author

jonnyka commented Jun 25, 2018

@rwky @dougwilson tests are added, readme updated
edit: checking the errors
edit #2: could it be an env issue? server and tests run fine on my env

  • osx
  • node 9.4.9
  • npm 5.6.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants