Follow-up from "Implement download feature for daily code coverage"
The following discussions from !27094 (merged) should be addressed:
-
@ayufan started a discussion: (+1 comment) Is this good permission to use? Are there permissions related to CI that should be used instead?
I see that we use
download_code
forGraphs
today. Maybe this is suggestion to future to have a correct permission:read_build_report_results
? -
@ayufan started a discussion: This seems to miss
project
. -
@ayufan started a discussion: @iamricecake Small hint, such methods can be defined at end to reduce effort required to scroll. This way we make at top what is important: how test is configured and executed, and at bottom a boilerplate :)