budgeting icon indicating copy to clipboard operation
budgeting copied to clipboard

Improve maintainability score to A

Open grgur opened this issue 6 years ago • 4 comments

Current maintainability score is B. Code climate shows how to improve that score here: https://codeclimate.com/github/ModusCreateOrg/budgeting

Let's get that A!

grgur avatar Sep 04 '18 08:09 grgur

one of the things required is a codeclimate config to ignore the flow-typed directory

nelsonomuto avatar Sep 22 '18 23:09 nelsonomuto

that reduces the number of files being analyzed from over 240 to 177.

then there are a few invalid "duplication" issues being raised, those need to be marked invalid; mostly with regards to the unit tests.

after this it reduces the amount of issues drastically

screen shot 2018-09-22 at 4 48 06 pm

nelsonomuto avatar Sep 22 '18 23:09 nelsonomuto

@grgur I don't have access to the Code Climate as a member so I can't mark anything as "invalid" but here is a link to my code climate report https://codeclimate.com/github/nelsonomuto/budgeting

Here's a PR to add the .codeclimate.yml config to exclude the flow-typed directory: https://github.com/ModusCreateOrg/budgeting/pull/201

nelsonomuto avatar Sep 22 '18 23:09 nelsonomuto

@nelsonomuto thank you so much for contributing! This gets us a lot closer to the target https://codeclimate.com/github/ModusCreateOrg/budgeting

I believe the other improvements fall into the fun re-org category :)

grgur avatar Sep 24 '18 07:09 grgur