Level / memdown

In-memory abstract-leveldown store for Node.js and browsers.

MIT License

287 stars 37 forks source link

Remove testing section from readme? #86

Closed vweevers closed 7 years ago

vweevers commented 7 years ago

Continuing from https://github.com/Level/memdown/issues/84#issuecomment-333413290. @ralphtheninja please elaborate :)

ralphtheninja commented 7 years ago

I want to reduce some of the information, mentioning npm test in a node.js README is being overly detailed, it's convention. The linter could be replaced by standard and also be part of the test. Maybe just mention how you can test locally in your browser and run coverage.

Just being nit-picky as always :wink:

vweevers commented 7 years ago

I want to reduce some of the information, mentioning npm test in a node.js README is being overly detailed, it's convention.

Agreed 👍

The linter could be replaced by standard

87.

and also be part of the test.

The current linter is indeed part of npm test so we can remove these instructions 👍