Skip to content

Prevent unnecessary validations during state changes

Gary Holtz requested to merge 408142-gmh-reduce-validation-calls into master

What does this MR do and why?

As part of #408142 (closed), this MR will skip a validation (validate_reviewer_size_length) when doing a state transition. The feature flag skip_validations_during_transitions has been added to add more validations/callbacks if necessary if this one doesn't improve performance.

MR acceptance checklist

Related issues

Related to #408142 (closed)

Testing

The MR includes unit tests for the new transitioning attribute and the validate_reviewer_size_length validation.

Backwards compatibility

The MR is backwards compatible.

Performance

The MR improves the performance of the application by skipping unnecessary validations during state changes.

Security

The MR does not introduce any security risks.

Maintainability

The MR improves the maintainability of the codebase by introducing a new transitioning attribute and skipping unnecessary validations during state changes.

This description was generated for revision 3bcb1933 using AI

Edited by Gary Holtz

Merge request reports