Add more tests - Githubissues

sindresorhus / dot-prop

Get, set, or delete a property from a nested object using a dot path

MIT License

812 stars 69 forks source link

Add more tests #70

Open sindresorhus opened 4 years ago

Richienb commented 3 years ago

@sindresorhus Could you add test coverage to the CI process?

sindresorhus commented 3 years ago

@Richienb All the coverage services I know about suck so much. Can't we just add c8 locally and get the coverage when running the tests locally? That should be enough.

Richienb commented 3 years ago

@sindresorhus Probably. What are some things you look for in a quality code coverage service?

sindresorhus commented 3 years ago

Reliability (The current ones have random failures all the time)
Stability (I had to change the config of the CIs multiple times)
Easy setup (I just want to add a line to a GitHub Actions workflow and be done)
Good and fast UI (The UI of the current ones are messy and very slow)