diff --git a/NEWS b/NEWS index 44ec7f0..57cd815 100644 --- a/NEWS +++ b/NEWS @@ -1,3 +1,11 @@ +CHANGES IN VERSION 1.48.0 +----------------------- + +BUG FIXES AND MINOR IMPROVEMENTS + + o Package tarball size check updated to have a max of 10 MB (@LiNk-NY, + #234). + CHANGES IN VERSION 1.46.0 ----------------------- diff --git a/R/BiocCheck.R b/R/BiocCheck.R index b231cfd..4c32430 100644 --- a/R/BiocCheck.R +++ b/R/BiocCheck.R @@ -205,7 +205,7 @@ BiocCheckRun <- if (is.null(dots[["no-check-pkg-size"]])){ handleCheck("Checking package size...") if (.BiocPackage$isTar){ - checkPackageSize(.BiocPackage, size=5) + checkPackageSize(.BiocPackage) } else { handleMessage("Skipped... only checked on source tarball", indent=4) } diff --git a/R/checks.R b/R/checks.R index 5c9fd18..2a3fa0b 100644 --- a/R/checks.R +++ b/R/checks.R @@ -28,13 +28,13 @@ checkDeprecatedPackages <- function(.BiocPackage) } } -checkPackageSize <- function(.BiocPackage, size=5){ +checkPackageSize <- function(.BiocPackage, size = 10) { pkg <- .BiocPackage$sourceDir pkgType <- .BiocPackage$packageType if (is.na(pkgType) || pkgType == "Software") { - maxSize <- size*10^6 ## 5MB + maxSize <- size*10^6 ## in MB pkgSize <- file.size(pkg) - if (pkgSize > maxSize){ + if (pkgSize > maxSize) { msgs <- c( paste0("Package Size: ", as.character(round(pkgSize/(10^6),2)), " MB"), diff --git a/vignettes/BiocCheck.Rmd b/vignettes/BiocCheck.Rmd index a1b697e..5b6aa77 100644 --- a/vignettes/BiocCheck.Rmd +++ b/vignettes/BiocCheck.Rmd @@ -156,7 +156,7 @@ Can be disabled with `--no-check-pkg-size` and `--no-check-file-size`. * **Checking package size** Checks that the package size meets _Bioconductor_ requirements. The current - package size limit is 5 MB for Software packages. Experiment Data and + package size limit is 10 MB for Software packages. Experiment Data and Annotation packages are excluded from this check. This check is only run if checking a source tarball. (ERROR)