-
-
Notifications
You must be signed in to change notification settings - Fork 114
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
[CHORE] improve codebase #270
Comments
@rohitdasu I would like to work on this |
Sure |
Please assign and let me know what revamps are needed @rohitdasu |
Sure, for now, could you remove unused packages and asset files(for example lottie animations). Once you done with the task please create a PR 🙌🏻 |
Thanks a lot @rohitdasu |
It's automatic, after merging your PR it will be added |
Removed lottie-react Unused dependencies
I believe some of them are getting used in config files or somewhere here & there. Could you please have a look @rohitdasu . |
Hi, postcss, prettier tailwind plugin, lint staged,useHooksts are required ones. Please check each dependency and it's usage, ex: Postcss is needed for TailwindCSS configuration |
Please raise a PR so that we can have better picture on it |
@rohitdasu |
Let's keep it as is so that we can have more PRs on this issue. I believe improving codebase is a never ending process 😅 |
Hi, @rohitdasu can I work on this issue? |
Sure go ahead @HKAgrwl |
Can you recommend me something I should get started with? |
could you check the issues that are present in this repo? if no issue is suitable for u than you can create one issue and work upon it. before working on the issue let us know. thanks |
@rohitdasu I think this issue is going to be open for a long time (as refactoring doesn't stop), can I also work on this issue? |
Sure, you can try to improve any piece of code |
I would like to join and contribute to improve the codebase, can I? @rohitdasu |
Sure |
Can I work on this |
Sure |
Description
requirement:
refactor code
implement best practices
enhance readability
The text was updated successfully, but these errors were encountered: