Do not perform Rubocop checks unless in CI and MR is not in draft
What does this MR do and why?
Do not perform check unless in CI and MR is not in draft
- from feedback here
- by doing this we'll remove the possibility of local danger runs catching this, but that should be ok as rubocop with lefthook should catch this unless the whole file is disabled for this cop and in that case we'll still see it, but just a little later after
draft
is removed on a fresh pipeline.
How to set up and validate locally
See reproduction in !138095 (comment 1670422112)
MR acceptance checklist
This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.
-
I have evaluated the MR acceptance checklist for this MR.
Related to #428157
Edited by Doug Stull