unhandled-error: use full function name in error message

mgechev / revive

🔥 ~6x faster, stricter, configurable, extensible, and beautiful drop-in replacement for golint

https://revive.run

MIT License

4.69k stars 269 forks source link

unhandled-error: use full function name in error message #962

Closed mmcloughlin closed 6 months ago

mmcloughlin commented 6 months ago

This PR updates the error message used by the unhandled-error rule to use the full function name. The full function name matches the name used to evaluate ignored functions in the configuration, therefore it's friendlier to users to wish to configure an exlusion based on an error message.

The error message is sufficiently covered by existing tests, which are updated in this PR.

Closes: #961

ccoVeille commented 4 months ago

Thank you for implemeting this, it would help a lot.