[_]: refactor/remove-dead-code #1822
Conversation
Deploying drive-web with
|
| Latest commit: |
950fe43
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://f2d29a4e.drive-web.pages.dev |
| Branch Preview URL: | https://refactor-remove-dead-code.drive-web.pages.dev |
|
|
WDYT @CandelR ? |
CandelR
left a comment
CandelR
left a comment
There was a problem hiding this comment.
@sg-gs I understand that it is a tool or AI that you have tried out to see how well it works for cleaning code, is that right?
It seems to do a good job of removing unused code. The only thing missing is some context about certain components. For example, in the Workspaces (B2B) section, there are some that are not used, but they were created at the time because there is a design, and I think that if B2B is continued, they will be used.
It also seems that it does not take into account orphaned translations.
But it does take into account tests of code that is not used, and deletes the code and the corresponding tests.
So it's not bad 👌
| onSavingAvatarError: (error: unknown) => void; | ||
| } | ||
|
|
||
| const UploadWorkspaceAvatarModal = ({ |
There was a problem hiding this comment.
@larry-internxt Do you remember why this component is not used?
| {!isUserTeams && isTeams && ( | ||
| <section className="flex h-40 w-full flex-col items-center justify-center rounded-xl border border-gray-10 bg-gray-1 py-10"> | ||
| <p className="font-regular text-center text-base text-gray-50"> | ||
| {t('preferences.workspace.members.tabs.teams.emptyTeams.title')} |
There was a problem hiding this comment.
nit: that translation seems unused after this component remove
|
As long as it is not currently needed, I will just wipe out that code. Once the moment arrives, we can rescue it from the corresponding PR, meanwhile, it is noise and contributes negatively to the coverage and maintenance of this project, I think @CandelR |
|
I am pretty sure I can wipe out another thousand lines, but I did not want to create a massive PR :) |
Description
Related Issues
Related Pull Requests
Checklist
Testing Process
Additional Notes