search

rohitdasu / projectmate

Supercharge your open-source contributions with projectmate.net; Discover open-source projects, connect with experienced maintainers, and collaborate with a community of passionate contributors. Join over 200+ registered users who are already making a difference
https://projectmate.net
MIT License
148 stars 107 forks source link

[CHORE] improve codebase #270

Open rohitdasu opened 1 year ago

rohitdasu commented 1 year ago

Description

requirement: refactor code implement best practices enhance readability

ashutosh887 commented 1 year ago

@rohitdasu I would like to work on this

rohitdasu commented 1 year ago

Sure

ashutosh887 commented 1 year ago

Please assign and let me know what revamps are needed @rohitdasu

rohitdasu commented 1 year ago

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 🙌🏻

ashutosh887 commented 1 year ago

Thanks a lot @rohitdasu How're the contributors section managed? Should I enlist myself there or it gets done automatically (just asking)

rohitdasu commented 1 year ago

Thanks a lot @rohitdasu

How're the contributors section managed? Should I enlist myself there or it gets done automatically (just asking)

It's automatic, after merging your PR it will be added

ashutosh887 commented 1 year ago

Removed lottie-react Apart from this, we have the following dependencies that can be removed:

Unused dependencies

I believe some of them are getting used in config files or somewhere here & there. Could you please have a look @rohitdasu .

rohitdasu commented 1 year ago

Removed lottie-react

Apart from this, we have the following dependencies that can be removed:

Unused dependencies

  • @hookform/resolvers

  • flowbite-react

  • next-themes

  • react-drag-drop-files

  • react-markdown

  • react-particles

  • react-quill

  • ts-node

  • tsparticles-preset-links

  • usehooks-ts

Unused devDependencies

  • @types/node

  • @types/uuid

  • autoprefixer

  • lint-staged

  • postcss

  • prettier-plugin-tailwindcss

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

rohitdasu commented 1 year ago

Please raise a PR so that we can have better picture on it

ashutosh887 commented 1 year ago

@rohitdasu This can be closed now

rohitdasu commented 1 year ago

@rohitdasu

This can be closed now

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 😅

HKAgrwl commented 1 year ago

Hi, @rohitdasu can I work on this issue?

ashutosh887 commented 1 year ago

Sure go ahead @HKAgrwl

HKAgrwl commented 1 year ago

Can you recommend me something I should get started with?

rohitdasu commented 1 year ago

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

muhammadbasitobaid commented 10 months ago

@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?

rohitdasu commented 10 months ago

@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

0xchikku commented 6 months ago

I would like to join and contribute to improve the codebase, can I? @rohitdasu

rohitdasu commented 6 months ago

I would like to join and contribute to improve the codebase, can I? @rohitdasu

Sure