Skip to content

Do not perform Rubocop checks unless in CI and MR is not in draft

Doug Stull requested to merge 428157-turn-off-for-draft into master

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.

Related to #428157

Edited by Doug Stull

Merge request reports