[COMPASS-4772] Address depcheck failures Created: 27/Apr/21 Updated: 28/Jun/23 Resolved: 11/Jan/23 |
|
| Status: | Closed |
| Project: | Compass |
| Component/s: | Tech debt |
| Affects Version/s: | None |
| Fix Version/s: | No version |
| Type: | Task | Priority: | Major - P3 |
| Reporter: | Sergey Petushkov | Assignee: | Unassigned |
| Resolution: | Won't Do | Votes: | 0 |
| Labels: | post-migration | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||
| Documentation Changes: | Not Needed | ||||
| Description |
|
We are replacing mongodb-js-precommit with depcheck + eslint for every imported package. Depcheck seems to be a better tool, but gives a different output and catches more issues, we know that whatever we currently have works, so there is no need to address those issues immediately, but post migration we should clean up depcheck usage and address the issues raised by the tool |
| Comments |
| Comment by Maurizio Casimirri [ 11/Jan/23 ] |
|
we are addressing this issue over time as we work on the individual packages |
| Comment by Maurizio Casimirri [ 21/Jun/21 ] |
|
We already re-enabled depcheck and fixed the "missing dependencies" errors. All the unused dependencies warning that where find are currently ignored by adding them to a .depcheckrc file. |