Closed
Description
The current lintr workflows for example here https://github.com/r-lib/actions/blob/v2-branch/examples/lint.yaml doesn't fail if a violation is found, which is counterintuive and differs from what normally CI/CD workflows do.
I think we should fail by default, and let people "ignore" the failure with an option. Unfortunately the guide here https://github.com/r-lib/lintr#continuous-integration is not very clear, I would have expected this to be the default