Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Frequency tests should consider current date #241

Open
yonjah opened this issue Jul 1, 2019 · 1 comment
Open

Frequency tests should consider current date #241

yonjah opened this issue Jul 1, 2019 · 1 comment

Comments

@yonjah
Copy link

yonjah commented Jul 1, 2019

I see some abandoned projects with only a few releases getting a maintainability score of 100

I think this is probably because evaluateReleasesFrequency and evaluateCommitsFrequency Only consider the time between commits / releases but do not actually consider the current date.
So if last release was two years ago a project can still have 100% maintainability score.
If on the other hand the same project would now fix some bug and add a new release they will actually loose their maintainability score.

If you consider the fact that forks and projects without any issue will get an issue distribution score of 0.7 where this score seem to average around 0.4 with some of the top maintained packages on npm a dead fork with a single release of any package can get a higher maintainability score than the original package

@yonjah
Copy link
Author

yonjah commented Jul 2, 2019

Sorry just noticed #231
Though it seem almost any package with v >= 1 will hit the finalize check especially if it doesn't have anyone using it to open any issues.

Not sure but maybe a check if a package had a couple of dozen closed issues before assuming it's been finalized will improve results ?

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant