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 6 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).