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

Prune some unused code + files #2308

Open
wants to merge 1 commit into
base: staging
Choose a base branch
from
Open

Prune some unused code + files #2308

wants to merge 1 commit into from

Conversation

yornaath
Copy link
Contributor

Pruning osme unused files and code using knip

@yornaath yornaath self-assigned this Feb 22, 2024
Copy link

vercel bot commented Feb 22, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
ui ✅ Ready (Inspect) Visit Preview Feb 22, 2024 1:15pm
1 Ignored Deployment
Name Status Preview Updated (UTC)
wsx ⬜️ Ignored (Inspect) Feb 22, 2024 1:15pm

Copy link
Contributor

@robhyrk robhyrk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems all good to remove. Maybe @Robiquet should have a look too.

Only thing I saw might be worth keeping was the local storage hook.

@@ -1,33 +0,0 @@
import { useState } from "react";
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

wondering if this might be good to keep for now? I see it was being used with the nfts

@robhyrk
Copy link
Contributor

robhyrk commented Feb 27, 2024

Also I'm wondering if maybe the yarn.lock should be ignored? I'm not entirely sure how that might affect the build process

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

Successfully merging this pull request may close these issues.

2 participants