Remove unused selectedLabelsList getter to fulfill eslint rule no-unused-property
-
Please check this box if this contribution uses AI-generated content (including content generated by GitLab Duo features) as outlined in the GitLab DCO & CLA. As a benefit of being a GitLab Community Contributor, you receive complimentary access to GitLab Duo.
What does this MR do and why?
Removes the unused selectedLabelsList Vuex getter from dropdown_contents_labels_view.vue. The getter was mapped but never used in the component.
This addresses one vue/no-unused-properties violation in app/assets/javascripts/sidebar/ as part of #525766 (closed) .
Created during the GitLab Meetup Hamburg hands-on session: https://www.meetup.com/gitlab-meetup-hamburg/events/311048113/ .
References
- Closes #525766 (closed)
Screenshots or screen recordings
N/A
How to set up and validate locally
- Run ESLint to verify no violations:
yarn eslint app/assets/javascripts/sidebar/components/labels/labels_select_vue/dropdown_contents_labels_view.vue - Verify file removed from ESLint todo list:
grep "dropdown_contents_labels_view.vue" .eslint_todo/vue-no-unused-properties.mjs
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.
MR Checklist (@gerardo-navarro)
-
Changelog entry added, if necessary -
Documentation created/updated via this MR -
Documentation reviewed by technical writer or follow-up review issue created -
Tests added for this feature/bug -
Tested in all supported browsers -
Conforms to the code review guidelines -
Conforms to the style guides -
Conforms to the javascript style guides -
Conforms to the database guides -
Conforms to the merge request performance guidelines -
No changelog needed (internal code quality improvement only) -
Existing tests cover the change (no new tests required) -
ESLint validation passes without warnings
Edited by Gerardo Navarro