r-lib / lintr

Static Code Analysis for R
https://lintr.r-lib.org
Other
1.2k stars 187 forks source link

Cleaner output in the examples on the website? #2632

Closed etiennebacher closed 4 months ago

etiennebacher commented 4 months ago

The examples on the website have an annoying prefix before them:

image

I might be missing something but it seems to me that this prefix is here to correctly mark lints in Github actions output: https://github.com/r-lib/lintr/blob/main/R/actions.R

I guess the issue is that the pkgdown workflow runs on Github actions so this printing method is enabled. However, I don't think it's useful to show this on the website. Maybe you could tweak in_github_actions() to be FALSE when IN_PKGDOWN is TRUE?

IndrajeetPatil commented 4 months ago

Good idea. See https://github.com/r-lib/lintr/pull/2633 :)