search

ddsol / speedtest.net

node.js SpeedTest.net client module
MIT License
611 stars 124 forks source link

Tidy documentation #80

Closed jsejcksn closed 5 years ago

jsejcksn commented 5 years ago

See #79

ddsol commented 5 years ago

Spelling of 'module' needs fixing. Also, if you don't mind, instead of

**`name`**_`: type`_

I think maybe

**`name`**: _`type`_

is easier to read and I'd appreciate if you could add this if possible perchance.

Besides that, this whole module could use some rewriting from scratch. :grinning:

jsejcksn commented 5 years ago

Done. Great catch on the spelling—thanks! 😅

Also, I just noticed: the nested list under the client property (in the data section) is not displaying correctly. Do you know why? It's formatted like the other two property lists...

ddsol commented 5 years ago

Yes, there are 2 spaces between the - and the **`client`**: so that's probably the issue.

jsejcksn commented 5 years ago

Should be all sorted now.