action #160023
closedUpdate eslint in qem-dashboard size:S
0%
Description
Motivation¶
dependabot already created an eslint update but tests failed, see https://github.com/openSUSE/qem-dashboard/pull/1082, now https://github.com/openSUSE/qem-dashboard/pull/1153
We should support dependabot
Acceptance criteria¶
- AC1: https://github.com/openSUSE/qem-dashboard/pull/1082 or the corresponding follow-up PRs are merged/closed and eslint updated
Suggestions¶
- Take a look into CI failures of https://github.com/openSUSE/qem-dashboard/pull/1082, fix the failures, apply the necessary JS style fixes as necessary, submit to the PR, get it merged
Updated by mkittler 10 months ago ยท Edited
The PR has been merged: https://github.com/openSUSE/qem-dashboard/pull/1153
The configuration for the import plugin is still disabled. Enabling it is blocked by https://github.com/import-js/eslint-plugin-import/issues/2948 and https://github.com/import-js/eslint-plugin-import/issues/2556. I don't think this plugin is very important so we might just consider this ticket resolved. Otherwise I'll keep it in blocked. Regardless of how we handle the progress ticket I have subscribed to the GitHub issues so if they are done I'll know and can bring back the import plugin (which should then be quite easy).