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

Rename length.path #395

Closed
nicolewhite opened this Issue Aug 28, 2016 · 2 comments

Comments

Projects
None yet
4 participants
@nicolewhite

nicolewhite commented Aug 28, 2016

Hi! You have an S3 method in your namespace defined here that's overriding the base::length function when I assign the class 'path' to an object.

See:

If it could be renamed / changed in some way, that would be great!

@pipemg

This comment has been minimized.

pipemg commented Nov 11, 2016

I have the same problem and others in the community of bioconductor

@JackieMium

This comment has been minimized.

JackieMium commented Jun 22, 2017

same problem here. might be related to this

@hadley hadley added the bug label Jul 24, 2017

@hadley hadley closed this in 1078b1d Jul 24, 2017

jiwalker-usgs added a commit to jiwalker-usgs/httr that referenced this issue Jul 24, 2017

jefferis added a commit to jefferis/RNeo4j that referenced this issue Mar 14, 2018

.onLoad replacing httr:::path.length no longer required
* r-lib/httr/issues/395 was closed in Jul 2017
* MAKE Check doesn't like this
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment