Adjust a few instances of `no-else-return`

We currently have `no-else-return` with `allowElseIf: true`. This fixes
a few instances to comply with `allowElseIf: false`.

ee/app/assets/javascripts/vue_shared/security_reports/components/dismissal_note.vue

@@ -63,9 +63,11 @@ export default {
           { pipelineLink, projectLink },
           false,
         );
-      } else if (pipelineLink && !projectLink) {
+      }
+      if (pipelineLink && !projectLink) {
         return sprintf(__('Dismissed on pipeline %{pipelineLink}'), { pipelineLink }, false);
-      } else if (!pipelineLink && projectLink) {
+      }
+      if (!pipelineLink && projectLink) {
         return sprintf(__('Dismissed at %{projectLink}'), { projectLink }, false);
       }
       return __('Dismissed');

ee/app/assets/javascripts/vue_shared/security_reports/mixins/reports_mixin.js

@@ -5,7 +5,8 @@ export default {
     checkReportStatus(loading, error) {
       if (loading) {
         return LOADING;
-      } else if (error) {
+      }
+      if (error) {
         return ERROR;
       }
 

ee/app/assets/javascripts/vue_shared/security_reports/store/utils.js

@@ -24,13 +24,15 @@ export const enrichVulnerabilityWithFeedback = (vulnerability, feedback = []) =>
           isDismissed: true,
           dismissalFeedback: fb,
         };
-      } else if (fb.feedback_type === 'issue' && fb.issue_iid) {
+      }
+      if (fb.feedback_type === 'issue' && fb.issue_iid) {
         return {
           ...vuln,
           hasIssue: true,
           issue_feedback: fb,
         };
-      } else if (fb.feedback_type === 'merge_request' && fb.merge_request_iid) {
+      }
+      if (fb.feedback_type === 'merge_request' && fb.merge_request_iid) {
         return {
           ...vuln,
           hasMergeRequest: true,