npms-io / npms-analyzer

The analyzer behind https://npms.io
MIT License
317 stars 38 forks source link

Integrate with Check Runs Github API #236

Open MichaelEPope opened 5 years ago

MichaelEPope commented 5 years ago

Hey, I just noticed that Travis-CI no longer uses the status Github API. They now use the Check Runs API (and I'm guessing some other CIs do that as well).

Would it be possible to get npms-analyzer to work with Check Runs as well as Status?

Here's a link to the announcement: https://blog.travis-ci.com/2018-09-27-deprecating-github-commit-status-api-for-github-apps-managed-repositories

astoilkov commented 3 years ago

I get incorrect quality values for my searches because of this. Only old modules that used the statuses API have a high score. So old modules have a higher score.

From what I understand from the source code the line here should be changed with check-runs instead of statuses.