[12.3] Remove 'needs: []' from '.gitlab/ci/review.gitlab-ci.yml'

What does this MR do?

This removes needs: [] which isn't actually supported and will throw an error after !19467 (merged) is merged.

Does this MR meet the acceptance criteria?

Conformity

• Code review guidelines
• Merge request performance guidelines
• Style guides
• Separation of EE specific content