Skip to content
This repository has been archived by the owner on Jun 1, 2020. It is now read-only.

Using dplyr::bind_rows() instead of dplyr::rbind_all() #1

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

romainfrancois
Copy link

We're tidyverse/dplyr#4579 in the process of releasing dplyr 0.8.4 and this package fail at our reverse dependency checks because the dplyr::rbind_all() function has been formally removed after being deprecated for many versions.

We might revert that decision because it affects a handful of packages, but in any case even if rbind_all() stays a little longer, you still should use bind_rows() instead.

* installing *source* package ‘Rariant’ ...
** using staged installation
** R
** inst
** byte-compile and prepare package for lazy loading
Warning messages:
1: package ‘GenomicRanges’ was built under R version 3.6.1
2: package ‘S4Vectors’ was built under R version 3.6.1
3: package ‘IRanges’ was built under R version 3.6.1
4: package ‘SummarizedExperiment’ was built under R version 3.6.1
5: package ‘BiocParallel’ was built under R version 3.6.1
6: package ‘Rsamtools’ was built under R version 3.6.1
Error: object 'rbind_all' is not exported by 'namespace:dplyr'
Execution halted

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
Require dplyr 0.5.0 for bind_rows()
Import bind_rows() instead of rbind_all()
using dplyr::bind_rows() instead of dplyr::rbind_all()
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant