All seem like reasonable suggestions to me. Currently the only place the
coverage is really visible is the badge we have on the main github page

https://github.com/apache/incubator-daffodil

I think all we need to do is delete/modify this config:

https://github.com/apache/incubator-daffodil/blob/master/.codecov.yml

And it looks like codecov has a bunch of configuration options for how
it should add comments to PRs. Whoeer make this change should probably
look through this and see if anything is worth changing in the default
config:

https://docs.codecov.io/docs/pull-request-comments




On 9/28/20 11:15 AM, Beckerle, Mike wrote:
> I'd like to turn back on emails coming from the automated coverage analysis.
> 
> Since I had no idea this was even still running automatically, I think more 
> noise from this it is not a distraction. At least that tells us if the 
> automated 
> checking still works, and a quick check of the email rules out major 
> regression 
> on these metrics.
> 
> I also think we should put links to these on the release notes pages. Each 
> release copies these pages from a prior and adapts those pages for the 
> release. 
> Having a section that links to the reports for the specific release would be 
> useful and serves as the reminder to study them a bit per release.
> 
> E.g, for 2.7.0 this pages is the specific coverage report for that release.
> 
> https://codecov.io/gh/apache/incubator-daffodil/commit/0ddaea2c29bbed35e2a4e62233545bb9357db68d/graphs/sunburst
> 
> However, I looked at the sonarcube page, but I couldn't figure out how to get 
> the sonarcube output page for a specific commit.
> 
> 
> 
> Mike Beckerle | Principal Engineer
> 
> OWL Cyber Defense
> 
> P +1-781-330-0412
> W owlcyberdefense.com <http://www.owlcyberdefense.com>
> 

Reply via email to