search

bestia-dev / cargo_crev_web

web server to query reviews from cargo-crev
https://lucianobestia.github.io/cargo_crev_web
MIT License
8 stars 0 forks source link

Newer reviewers are missing #22

Closed Minoru closed 2 years ago

Minoru commented 3 years ago

https://web.crev.dev/rust-reviews/reviewers/ only lists 64 reviewers, but it looks like there should be more:

$ cargo crev id query all | wc -l
94

As a second data point, I've been in auto-crev-proofs for almost a week now (ignore trust: medium — that has been fixed), but I'm still missing from web.crev.dev.

Perhaps a cron job failed somewhere?

bestia-dev commented 3 years ago

I will check it now. Thanks. There is a lot of users that install crev and create a repository, but then never add a single review. I blocklist those until something changes. You can find them here: https://web.crev.dev/rust-reviews/reserved_folder/blocklisted_repos/

Minoru commented 3 years ago

Ah, got it! My repo, https://github.com/Minoru/crev-proofs, is not in this list, so probably something else is going on.

bestia-dev commented 3 years ago

I can find you: in number 15 of https://web.crev.dev/rust-reviews/reviewers/

bestia-dev commented 3 years ago

It looks that upper-case first character are sorted before lower-case. Sorry for that. I will try to correct that. Thanks to pointing that. I appreciate.

Minoru commented 3 years ago

I wonder how I managed to miss myself there >_< Probably ".git" confused me. I should've just used page search in the browser… Thank you, @LucianoBestia!