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.

Sign up for GitHub

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

Jump to bottom

request: add guess_integer as arg for type_convert() #1264

Closed
jmbarbone opened this issue Aug 5, 2021 · 0 comments · Fixed by #1265
Closed

request: add guess_integer as arg for type_convert() #1264

jmbarbone opened this issue Aug 5, 2021 · 0 comments · Fixed by #1265

Comments

@jmbarbone
Copy link
Contributor

type_convert implements all args from guess_parser expect for guess_integers:

readr/R/type_convert.R

Line 53 in 5b7408e

guesses <- lapply(char_cols, guess_parser, locale = locale, na = na)

@jmbarbone jmbarbone changed the title request: add guess_integer as arg for type_convert request: add guess_integer as arg for type_convert() Aug 5, 2021
@jmbarbone jmbarbone mentioned this issue Aug 5, 2021
Merged
jimhester pushed a commit that referenced this issue Aug 6, 2021
@jmbarbone
Include guess_integer argument in type_convert
53813a8 
Fixes #1264
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Include guess_integer argument in type_convert jmbarbone/readr
1 participant
@jmbarbone