Skip to content

Conversation

@abmusse
Copy link
Member

@abmusse abmusse commented Apr 22, 2020

Resolves #257

@abmusse abmusse requested a review from kadler April 22, 2020 16:28
@abmusse
Copy link
Member Author

abmusse commented Apr 22, 2020

FYI Moved the transport information to https://nodejs-itoolkit.readthedocs.io/en/latest/Connection.html#transports

@kadler
Copy link
Member

kadler commented Apr 22, 2020

Definitely good removals, but now it seems very barren. I'd like to see some basic intro info, like what we have on the docs intro page. eg. more info on what it does, why it's useful, etc.

https://medium.com/@meakaakka/a-beginners-guide-to-writing-a-kickass-readme-7ac01da88ab3

@abmusse abmusse force-pushed the docs-simplify-readme branch from 58c856e to 177e4f0 Compare April 23, 2020 17:38
@abmusse
Copy link
Member Author

abmusse commented Apr 23, 2020

Need to resolve a simple conflict:

image

@abmusse
Copy link
Member Author

abmusse commented Apr 24, 2020

@kadler This is ready for re-review

@kadler kadler merged commit 85c860b into master Apr 24, 2020
@kadler kadler deleted the docs-simplify-readme branch April 24, 2020 15:22
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.

Simplify readme

3 participants