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

Repo standard/code quality analysis #433

Open
SoulPancake opened this issue Jun 26, 2024 · 0 comments
Open

Repo standard/code quality analysis #433

SoulPancake opened this issue Jun 26, 2024 · 0 comments

Comments

@SoulPancake
Copy link

I think this is a very early to comment on this but perhaps there's a lot of work to be done to make this repo better.

I was only going through the code, one of the very first things that comes as an issue is

  1. The incredibly large readme.md
    It's overwhelming to have all that info altogether at one place, instead it should be modularized to point to a bunch of separate docs describing the installation steps or other things, maybe links in some cases.
    It probably seems useful to have all the info at one place but it is not dev-friendly at all.

  2. Linters please?
    I don't mean to be a snob but without a linter eventually the debt of not maintaining code style can catch up, even though this is possibly only maintained by few people at this point, avoiding this as early as possible is a life-saver.
    image

  3. Straight up tests not working
    image
    image

Apart from having incredibly low coverage, the repo also has some not-working tests.
Having properly tested code will definitely ensure low tech debt over-time.

I understand a lot of this might be tried/tested code and under maintenance but coming from working in very large repos with a significant number of contributors, hope all these points help out to be a large number of
good first issues for the community.

I'm down to contribute to few of these and also help setting up automated checks for these to maintain quality, in this repo as well as others.

Cheers and good luck!
Long live OpenSource.

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

No branches or pull requests

1 participant