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

Remove all the unused variables #12

Open
BenzeneAlcohol opened this issue Nov 14, 2023 · 2 comments · May be fixed by #13
Open

Remove all the unused variables #12

BenzeneAlcohol opened this issue Nov 14, 2023 · 2 comments · May be fixed by #13
Labels
good first issue Good for newcomers

Comments

@BenzeneAlcohol
Copy link
Collaborator

There are various places in the code where variables are defined but never used.

Eg:

const location = useLocation();

import { motion, AnimatePresence } from "framer-motion";

Whenever you run the application, you get the warning in the console and you can use it to find the places where it happens.
All of this needs to be removed/handled appropriately,

@BenzeneAlcohol BenzeneAlcohol added the good first issue Good for newcomers label Nov 14, 2023
@j-hariharan
Copy link

While at it, it would be a good idea to change the build tool to Vite from create-react-app to remove all the dependency issues and severe warnings generated when running npm install in the repository. Vite is also a faster and more modern build tool compared to create-react-app.

@BenzeneAlcohol
Copy link
Collaborator Author

We could try that definitely. Can you make a detailed document about this? Indicating why you want to migrate, and how you plan to do it. It will give us a better direction and sense.

@BenzeneAlcohol BenzeneAlcohol linked a pull request Dec 27, 2023 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants