Skip to content

Removal of more backend Visual Review code

drew stachon requested to merge 387751-deprecation-visual-review-be into master

What does this MR do and why?

This removes a bunch of dead code that's no longer used as of Remove visual review tools (!149586 - merged)

There was more removal than I thought, so we removed interfaces and anything user-facing in the first MR, and are sending follow-ups to do more removal behind the scenes.

MR acceptance checklist

Please evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.

How to set up and validate locally

Just run the test suite. All the changes in here are removals, so there's no additive thing to go look at. If everything else still works, we're good.

Related to #387751

Edited by drew stachon

Merge request reports