Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

covr::codecov() fails on a package without functions #427

Closed
lcolladotor opened this issue Apr 14, 2020 · 3 comments
Closed

covr::codecov() fails on a package without functions #427

lcolladotor opened this issue Apr 14, 2020 · 3 comments

Comments

@lcolladotor
Copy link

Hi,

I'm just starting a new package and setting things up (first time using GitHub actions) and ran into an error which I assume is simply because the package currently has no functions yet. This is related #57.

Anyway, it's a simple error but the error message could be confusing. Hence why I'm reporting it.

Best,
Leo

Error reproduced locally

> covr::codecov()
Error in aggregate.data.frame(mf[1L], mf[-1L], FUN = FUN, ...) : 
  no rows to aggregate
> traceback()
11: stop("no rows to aggregate")
10: aggregate.data.frame(mf[1L], mf[-1L], FUN = FUN, ...)
9: aggregate.formula(value ~ ., x, sum)
8: aggregate(value ~ ., x, sum)
7: merge_values(df)
6: as.data.frame.coverage(coverage)
5: as.data.frame(coverage)
4: per_line(x)
3: lapply(per_line(x), function(xx) {
       xx$coverage <- c(NA, xx$coverage)
       xx
   })
2: to_codecov(coverage)
1: covr::codecov()

Local R session information

> library('sessioninfo')
> options(width = 120)
> session_info()
─ Session info ───────────────────────────────────────────────────────────────────────────────────────────────────────
 setting  value                       
 version  R version 3.6.3 (2020-02-29)
 os       macOS Catalina 10.15.4      
 system   x86_64, darwin15.6.0        
 ui       RStudio                     
 language (EN)                        
 collate  en_US.UTF-8                 
 ctype    en_US.UTF-8                 
 tz       America/New_York            
 date     2020-04-14Packages ───────────────────────────────────────────────────────────────────────────────────────────────────────────
 package     * version date       lib source                            
 assertthat    0.2.1   2019-03-21 [1] CRAN (R 3.6.0)                    
 backports     1.1.6   2020-04-05 [1] CRAN (R 3.6.3)                    
 callr         3.4.3   2020-03-28 [1] CRAN (R 3.6.2)                    
 cli           2.0.2   2020-02-28 [1] CRAN (R 3.6.0)                    
 colorout    * 1.2-1   2019-05-07 [1] Github (jalvesaq/colorout@7ea9440)
 covr          3.5.0   2020-03-06 [1] CRAN (R 3.6.0)                    
 crayon        1.3.4   2017-09-16 [1] CRAN (R 3.6.0)                    
 desc          1.2.0   2018-05-01 [1] CRAN (R 3.6.0)                    
 devtools    * 2.3.0   2020-04-10 [1] CRAN (R 3.6.3)                    
 digest        0.6.25  2020-02-23 [1] CRAN (R 3.6.0)                    
 ellipsis      0.3.0   2019-09-20 [1] CRAN (R 3.6.0)                    
 fansi         0.4.1   2020-01-08 [1] CRAN (R 3.6.0)                    
 fs            1.4.1   2020-04-04 [1] CRAN (R 3.6.3)                    
 glue          1.4.0   2020-04-03 [1] CRAN (R 3.6.3)                    
 lazyeval      0.2.2   2019-03-15 [1] CRAN (R 3.6.0)                    
 magrittr      1.5     2014-11-22 [1] CRAN (R 3.6.0)                    
 memoise       1.1.0   2017-04-21 [1] CRAN (R 3.6.0)                    
 packrat       0.5.0   2018-11-14 [1] CRAN (R 3.6.0)                    
 pkgbuild      1.0.6   2019-10-09 [1] CRAN (R 3.6.0)                    
 pkgload       1.0.2   2018-10-29 [1] CRAN (R 3.6.0)                    
 prettyunits   1.1.1   2020-01-24 [1] CRAN (R 3.6.2)                    
 processx      3.4.2   2020-02-09 [1] CRAN (R 3.6.0)                    
 ps            1.3.2   2020-02-13 [1] CRAN (R 3.6.0)                    
 R6            2.4.1   2019-11-12 [1] CRAN (R 3.6.1)                    
 remotes       2.1.1   2020-02-15 [1] CRAN (R 3.6.0)                    
 rex           1.1.2   2017-10-19 [1] CRAN (R 3.6.0)                    
 rlang         0.4.5   2020-03-01 [1] CRAN (R 3.6.0)                    
 rprojroot     1.3-2   2018-01-03 [1] CRAN (R 3.6.0)                    
 rstudioapi    0.11    2020-02-07 [1] CRAN (R 3.6.0)                    
 sessioninfo * 1.1.1   2018-11-05 [1] CRAN (R 3.6.0)                    
 testthat    * 2.3.2   2020-03-02 [1] CRAN (R 3.6.2)                    
 usethis     * 1.6.0   2020-04-09 [1] CRAN (R 3.6.3)                    
 withr         2.1.2   2018-03-15 [1] CRAN (R 3.6.0)                    
 yaml          2.2.1   2020-02-01 [1] CRAN (R 3.6.0)                    

[1] /Library/Frameworks/R.framework/Versions/3.6/Resources/library
lcolladotor added a commit to LieberInstitute/recount3 that referenced this issue Apr 17, 2020
@lcolladotor
Copy link
Author

At LieberInstitute/recount3@108ade7 I did something similar to what you did already for one of your repos r-lib/actions@331cc49 (I just noticed we did nearly the same thing hehe)

@jimhester
Copy link
Member

While it would be nice to handle this use case I think it is not common enough to worry about for now.

lcolladotor added a commit to lcolladotor/biocthis that referenced this issue May 9, 2020
Created both an introductory and a developer's notes vignette, updated README
and docs with examples, added a second biocViews term, fixed some small
bugs/typos.

Related links (as many as I could remember):

* https://rstd.io/tidytools19
* https://twitter.com/CVWickham
* https://twitter.com/hadleywickham
* https://www.rstudio.com/products/rstudio/download
* https://comunidadbioinfo.github.io/post/building-tidy-tools-cdsb-runconf-2019/#.XrbLMxNKiu4
* http://bioconductor.org/
* https://lcolladotor.github.io/pkgs/
* https://stat.ethz.ch/pipermail/bioc-devel/2020-March/016365.html
* https://www.bioconductor.org/help/docker/
* https://stat.ethz.ch/pipermail/bioc-devel/2020-April/016532.html
* https://github.com/features/actions
* https://stat.ethz.ch/pipermail/bioc-devel/2020-April/016650.html
* r-lib/actions#84
* r-lib/usethis#1108
* r-lib/styler#636
* Bioconductor/BiocCheck#57
* Bioconductor/bioconductor.org#54
* http://bioconductor.org/developers/how-to/coding-style/
* https://style.tidyverse.org/
* https://twitter.com/lorenzwalthert
* https://twitter.com/mt_morgan
* https://docs.travis-ci.com/user/languages/r/
* r-lib/pkgdown#1206
* r-lib/pkgdown#1230
* https://twitter.com/jimhester_
* https://www.jimhester.com/talk/2020-rsc-github-actions/
* https://github.com/Bioconductor/BBS
* https://github.com/Bioconductor/packagebuilder
* https://www.appveyor.com/
* r-hub/rhub#52
* r-hub/rhub#38
* https://www.tidyverse.org/blog/2020/04/usethis-1-6-0/
* https://github.com/r-lib/actions/tree/master/examples
* https://yihui.org/en/2018/03/second-pull-request/
* https://github.com/r-lib/actions/blob/master/examples/check-standard.yaml
* https://help.github.com/en/actions
* https://ropenscilabs.github.io/actions_sandbox/
* https://twitter.com/seandavis12
* https://github.com/seandavi/BiocActions/blob/master/.github/workflows/main.yml
* https://twitter.com/CSoneson
* https://github.com/csoneson/dreval/blob/master/.github/workflows/R-CMD-check.yaml
* https://bioc-community.herokuapp.com/
* https://github.com/leekgroup/derfinderPlot/blob/master/.github/workflows/check-bioc.yml
* https://github.com/LieberInstitute/recount3/blob/master/.github/workflows/check-bioc.yml
* https://github.com/hpages
* r-lib/actions#68
* r-lib/actions#85
* https://twitter.com/opencpu
* https://community.rstudio.com/u/const-ae
* https://community.rstudio.com/t/compiler-support-fo-c-14-features-on-windows/57284/4
* r-lib/xml2#296
* r-lib/xml2#302
* https://github.com/r-lib/usethis/blob/master/.github/workflows/R-CMD-check.yaml
* https://github.com/r-lib/usethis/commits/master/.github/workflows/R-CMD-check.yaml
* https://stat.ethz.ch/pipermail/bioc-devel/2020-April/016703.html
* https://stat.ethz.ch/pipermail/bioc-devel/2020-April/thread.html
* r-lib/remotes#296
* r-lib/actions#86
* r-lib/covr#427
* https://github.com/r-lib/actions/blob/master/examples/pr-commands.yaml
* https://www.digitalocean.com/community/tutorials/how-to-install-git-on-ubuntu-18-04
* r-lib/actions#50
* actions/checkout#238
* https://github.com/rocker-org/rocker-versioned2/blob/master/dockerfiles/Dockerfile_rstudio_4.0.0-ubuntu18.04
* https://twitter.com/niteshturaga
* https://twitter.com/cboettig
* rocker-org/rocker-versioned#208
* https://github.community/t5/GitHub-Actions/bd-p/actions
* https://www.r-consortium.org/blog/2020/03/18/cdsb-diversity-and-outreach-hotspot-in-mexico
* https://github.com/maxheld83
* r-lib/actions#87
* https://github.com/yutannihilation
@stefanoborini
Copy link

@jimhester when there's a bug, the process normally is not to close it as "we don't care". A bug is a bug and must be fixed, and this is a bug. Leave it open at least.

@jimhester jimhester reopened this Jan 20, 2021
evanamiesgalonski added a commit to poissonconsulting/kootlake that referenced this issue Oct 8, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants