Bioconductor / BiocCheck

http://bioconductor.org/packages/BiocCheck
8 stars 26 forks source link

bioc-devel mailing list check still not always accurate #205

Closed lshep closed 8 months ago

lshep commented 8 months ago

I can confirm the single package builder is using version 1.39.10 See mailing list https://stat.ethz.ch/pipermail/bioc-devel/2024-January/020119.html for email = "richardheery@gmail.com".

I can confirm the email is registered on bioc-devel but it is report not. http://staging.bioconductor.org:8000/job/19968/

lshep commented 8 months ago

also someone else this morning: https://github.com/Bioconductor/Contributions/issues/3076#issuecomment-1881357053

LiNk-NY commented 8 months ago

I ran this locally and I didn't get the error:

> checkForBiocDevelSubscription("~/bioc/methodical/")
    Maintainer is subscribed to bioc-devel.
> grep("Version:", readLines("~/bioc/methodical/DESCRIPTION"), fixed = TRUE, value = TRUE)
[1] "Version: 0.99.20"

and for mobileRNA

> checkForBiocDevelSubscription("~/bioc/mobileRNA/")
    Maintainer is subscribed to bioc-devel.
lshep commented 8 months ago

Well they keep coming in. Here is another one that was passing but now is not:
https://github.com/Bioconductor/Contributions/issues/3251#issuecomment-1882506846

LiNk-NY commented 8 months ago

Thanks, it looks like the bug was only happening with Maintainer fields in the tarball. It should be resolved in 979e0d8c3a554243d10e88520ac9ed622a8993df