Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

MTV-1988: Knip - Remove unused files #1

Draft
wants to merge 1 commit into
base: mtv-1988
Choose a base branch
from

Conversation

jschuler
Copy link
Owner

@jschuler jschuler commented Feb 27, 2025

📝 Links

Merge after kubev2v#1498
Once the dependent PR is merged I will rebase this PR against upstream main

Towards https://issues.redhat.com/browse/MTV-1988

📝 Description

Remove unused files with the help of knip

Before

yarn knip --files

Unused files (23)
packages/forklift-console-plugin/src/index.ts
packages/forklift-console-plugin/src/modules/Migrations/dynamic-plugin.ts
packages/forklift-console-plugin/src/modules/Migrations/index.ts
packages/forklift-console-plugin/src/modules/NetworkMaps/index.ts
packages/forklift-console-plugin/src/modules/NetworkMaps/views/details/index.ts
packages/forklift-console-plugin/src/modules/NetworkMaps/views/index.ts
packages/forklift-console-plugin/src/modules/NetworkMaps/views/list/index.ts
packages/forklift-console-plugin/src/modules/Overview/views/overview/tabs/Pods/cards/index.ts
packages/forklift-console-plugin/src/modules/Overview/views/overview/tabs/Pods/cards/PodsCard.tsx
packages/forklift-console-plugin/src/modules/Overview/views/overview/tabs/Pods/ForkliftControllerPodsTab.tsx
packages/forklift-console-plugin/src/modules/Overview/views/overview/tabs/Pods/index.ts
packages/forklift-console-plugin/src/modules/Plans/index.ts
packages/forklift-console-plugin/src/modules/Plans/views/create/constanats/index.ts
packages/forklift-console-plugin/src/modules/Plans/views/create/index.ts
packages/forklift-console-plugin/src/modules/Plans/views/create/steps/CreateMigrationPlan/index.ts
packages/forklift-console-plugin/src/modules/Plans/views/index.ts
packages/forklift-console-plugin/src/modules/Plans/views/list/components/StatusCell.tsx
packages/forklift-console-plugin/src/modules/Plans/views/list/components/VMsProgressCell.tsx
packages/forklift-console-plugin/src/modules/Providers/views/migrate/components/EditableDescriptionItem.tsx
packages/forklift-console-plugin/src/modules/StorageMaps/index.ts
packages/forklift-console-plugin/src/modules/StorageMaps/views/details/index.ts
packages/forklift-console-plugin/src/modules/StorageMaps/views/index.ts
packages/forklift-console-plugin/src/modules/StorageMaps/views/list/index.ts

After

yarn knip --files
yarn run v1.22.22
$ knip --files
✂️  Excellent, Knip found no issues.

🎥 Demo

Please add a video or an image of the behavior/changes

📝 CC://

@tag as needed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant