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 okurz about 1 month ago
- Target version changed from Tools - Next to Ready
Updated by livdywan about 1 month ago
- Subject changed from Update eslint in qem-dashboard to Update eslint in qem-dashboard size:S
- Description updated (diff)
- Status changed from New to Workable
Updated by mkittler about 1 month ago
- Status changed from Workable to Feedback
Updated by mkittler about 1 month 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).
Updated by mkittler about 1 month ago
- Status changed from Feedback to Resolved
In the meeting today there were no clear preferences so I'm considering this resolved for now.