Bug #48258
closedmgr/dashboard: Switch from tslint to eslint
0%
Description
tslint is deprecated, so the Dashboard should switch to eslint.
See https://tracker.ceph.com/issues/48258#note-4
Adi went deeper into this: https://docs.google.com/document/d/1gs5lUkIyT5dGoVW1nGloZRFzEEl57D3p6DGBLX7zY1Y/edit
Updated by Ernesto Puerta over 3 years ago
Good finding, thanks Volker! Any clue on how easy/hard would this be to do?
Updated by Ernesto Puerta over 3 years ago
- Status changed from New to Triaged
- Priority changed from Normal to Low
Updated by Ernesto Puerta about 3 years ago
- Project changed from mgr to Dashboard
- Category changed from 165 to UI
Updated by Alfonso Martínez almost 3 years ago
- Assignee changed from Volker Theile to Alfonso Martínez
We should follow this guide:
https://github.com/angular-eslint/angular-eslint#migrating-an-angular-cli-project-from-codelyzer-and-tslint
Updated by Ernesto Puerta over 2 years ago
- Translation missing: en.field_tag_list set to dashboard_sprint_23
Updated by Kefu Chai over 2 years ago
- Status changed from Triaged to Fix Under Review
- Pull request ID set to 43480
Updated by Ernesto Puerta almost 2 years ago
Alfonso Martínez wrote:
We should follow this guide:
https://github.com/angular-eslint/angular-eslint#migrating-an-angular-cli-project-from-codelyzer-and-tslint
A tool for automating that:
https://github.com/typescript-eslint/tslint-to-eslint-config
There's another PR trying this: https://github.com/ceph/ceph/pull/45767
Updated by Nizamudeen A over 1 year ago
great, thanks Sedrick. you can assign it to you. There are two PRs opened currently. You can go over the discussions inside them and continue with it https://github.com/ceph/ceph/pull/43480, https://github.com/ceph/ceph/pull/45767. It'll give you some ideas.
Updated by Ngwa Sedrick Meh over 1 year ago
- Assignee changed from Alfonso Martínez to Ngwa Sedrick Meh
Updated by Nizamudeen A over 1 year ago
- Status changed from Fix Under Review to Pending Backport
- Assignee changed from Ngwa Sedrick Meh to Nizamudeen A
- Pull request ID changed from 43480 to 48569
Updated by Nizamudeen A about 1 year ago
- Status changed from Pending Backport to Fix Under Review
Updated by Nizamudeen A about 1 year ago
- Status changed from Fix Under Review to Pending Backport
- Tags deleted (
backport_processed)
Updated by Backport Bot about 1 year ago
- Copied to Backport #58730: quincy: mgr/dashboard: Switch from tslint to eslint added
Updated by Nizamudeen A about 1 year ago
- Status changed from Pending Backport to Resolved