Remove needs clause from review job

We don't actually need to have needs for this job, we can just let it run in normal stage order. The deploy stage is the last stage, so it'll only run when everything else has completed, which makes sense.

After the change in the related MR below, this needs entry was causing invalid YAML for pipelines triggered by upstream branches, because the compile_dev job no longer runs in those pipelines. Ex: https://gitlab.com/gitlab-org/gitlab-docs/-/pipelines/703054555

Related to !3292 (merged)