Utilise 'needs:' for review-qa-smoke
What does this MR do?
Since #31264 (closed) got resolved, we can now utilise needs:
for review-qa-job
to depend on review-deploy
. It will help to avoid failures when Review App is not deployed successfully but smoke tests are still attempted.
See #323999 (closed) & gitlab-com/www-gitlab-com#11568 (closed) for more information.
Does this MR meet the acceptance criteria?
Conformity
-
📋 Does this MR need a changelog?-
I have included a changelog entry. -
I have not included a changelog entry because _____.
-
-
Documentation (if required) -
Code review guidelines -
Merge request performance guidelines -
Style guides -
Database guides -
Separation of EE specific content