search

ropensci-review-tools / roreviewapi

API endpoint for package reviews via ropensci-review-bot
https://docs.ropensci.org/roreviewapi
4 stars 1 forks source link

Revert to including srr reports as part of review checks #62

Closed noamross closed 1 month ago

noamross commented 1 month ago

We previously included reports of srr standards compliance directly in the automated reports (e.g., https://github.com/ropensci/software-review/issues/603#issuecomment-1689418954). These were quite helpful because it gave a way for statistical reviewers to get a sense of the compliance approach before they had to download anything to run on their own machine. Can we return to this?

mpadge commented 1 month ago

So it seems that this ability has always existed, and still does. I just don't know what happened with https://github.com/ropensci/software-review/issues/615#issuecomment-1772393481. The report was generated, and can be seen here, but for some reason at that time the line linking to that did not appear in the checks report.

Anyway, if checks are re-run now, they definitely include link to the report, so this can be closed. Thanks!