Skip to content

Remove ternary expression for component

What does this MR do and why?

vue/no-unused-components rule does not seem to work in ee/app/assets/javascripts/security_dashboard/components/shared/vulnerability_details_graphql/index.vue because of the ternary expression inside the :is of <component /> as pointed out here.

Fix by simply using v-if / v-else in the template. After the change, the rule is correctly working again. See screenshot for when <gl-friendly-wrap> is commented out.

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.

Screenshots or screen recordings

Screenshot_2024-02-29_at_16.03.49

Screenshot_2024-02-29_at_16.03.29

How to set up and validate locally

Merge request reports