Good day I ran ESLint and found 100+ unused imports across many files. Should I:
-
Remove all of them in one PR?
-
Start with a specific directory/subset?
-
Focus on specific files you’ve identified?
Kindly Guide @tasksolver @Agaba_Derrick_Junior @Ariho-Seth @Moses_Mutesasira @reagan
Hello @Leone_Chirodza , what issue are you working on?
Remove unused imports for the frontend. #1554
1 Like
Wow, great @Leone_Chirodza . Start small, and scale up. I mean, start with a few files(like 5 depending on the number of files). This would ensure that you easily manage and track changes and also would ensure seamless troubleshooting.
1 Like
Removed unused React imports from multiple components (ConfirmPopup, CustomCheckBox, CustomSelect, GlobalSideBar, PageBreadCrumb, Utils). Code builds successfully; no functional impact. Waiting for the maintainers to review the pull request
1 Like
@Leone_Chirodza thanks for the PR I have been able to add some comments, you can as well address them
1 Like
can we also fix the excessive eslint warnings in the frontend container logs which make debugging difficult.
3 Likes