search

undercover-el / undercover.el

A test coverage library for Emacs
MIT License
86 stars 14 forks source link

Refactor away the need for ignore-errors #49

Open CyberShadow opened 4 years ago

CyberShadow commented 4 years ago

Ignoring all errors may hide important information from users / developers.

https://github.com/undercover-el/undercover.el/pull/45 fixes one instance of this (though, with-demoted-errors is not a complete fix).

As for the other remaining instance, the code seems to currently rely on it (i.e. errors occur as part of normal functioning). The code should be updated to handle error conditions explicitly in the place where they may occur; then, ignore-errors can be removed.

CyberShadow commented 3 years ago

shut-up is another thing that we could probably use less of.