Fix useless dynamic definition in push rules helper (!117814) · Merge requests · GitLab.org / GitLab

charlie ablett requested to merge 406761-cablett-push-rule-spec into master Apr 18, 2023

What does this MR do and why?

Fixes an offense for Rspec/UselessDynamicDefinition in PushRulesHelper. No user-facing changes.

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 #406761 (closed)

Edited Apr 18, 2023 by charlie ablett

Fix useless dynamic definition in push rules helper

What does this MR do and why?

MR acceptance checklist

Merge request reports