gojp / goreportcard

A report card for your Go application
https://goreportcard.com
Apache License 2.0
2.01k stars 249 forks source link

Failed processes shouldn't affect rating #339

Closed lmorg closed 3 years ago

lmorg commented 3 years ago

goreportcard appears to shell out to a number of 3rd party ELFs, but if those fail you get a big fat 0% against that metric and your score dropping significantly even though there might not be any issue.

The kind of failures I'm describing are:

An error occurred while running this test (exit status 2)

Failed processes should either return 100% or, better yet, not be included in the calculation for the project rating.

lmorg commented 3 years ago

To give an example of what I mean: image