Skip to content

Conversation

@cprussin
Copy link

@cprussin cprussin commented Aug 2, 2017

I just realized that having the most important function (createServer) last makes the docs crazy hard to read. This PR just moves the createServer function to the top of the file, so it's the first function in the docs for Node.HTTP.Secure.

@hdgarrood
Copy link
Member

Funnily enough I was just working on addressing this in the compiler: purescript/purescript#3003

Anyway 👍, although perhaps it would make sense to have createServer before SSLOptions too?

@cprussin
Copy link
Author

cprussin commented Aug 2, 2017

I agree, done, thanks

@hdgarrood
Copy link
Member

Thanks!

@hdgarrood hdgarrood merged commit 1d3890c into purescript-node:master Aug 2, 2017
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