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

fread autostart removal revdep easycsv example error #6714

Open
tdhock opened this issue Jan 9, 2025 · 4 comments
Open

fread autostart removal revdep easycsv example error #6714

tdhock opened this issue Jan 9, 2025 · 4 comments
Assignees
Labels
fread revdep Reverse dependencies WAI Working as intended
Milestone

Comments

@tdhock
Copy link
Member

tdhock commented Jan 9, 2025

revdep check machine says
#6653 broke easycsv

* checking examples ... ERROR
Running examples in 'easycsv-Ex.R' failed
The error most likely occurred in:

> base::assign(".ptime", proc.time(), pos = "CheckExEnv")
> ### Name: fread_zip
> ### Title: read multiple csv files into named data frames
> ### Aliases: fread_zip
> ### Keywords: ~utilities ~misc
> 
> ### ** Examples
> 
> 
>   require(easycsv)
>   filezip <- system.file("exampleZips", "example_tables.zip", package="easycsv")
>   fread_zip(filezip)
Error in data.table::fread(input = tbl, sep = sep, nrows = nrows, header = header,  : 
  'autostart' is deprecated. Consider skip='string' or skip=n. This argument will be removed in the next release.
Calls: fread_zip -> <Anonymous> -> stopf -> raise_condition -> signal
Execution halted

I filed bogind/easycsv#7

@tdhock tdhock self-assigned this Jan 9, 2025
@tdhock tdhock added fread revdep Reverse dependencies labels Jan 9, 2025
@tdhock tdhock added this to the 1.17.0 milestone Jan 10, 2025
@MichaelChirico
Copy link
Member

Closing PR: bogind/easycsv#8

@tdhock tdhock changed the title fread autostart removal revdep example error fread autostart removal revdep easycsv example error Jan 20, 2025
@MichaelChirico MichaelChirico added the WAI Working as intended label Jan 23, 2025
@TysonStanley
Copy link
Member

Not sure that one will be merged soon (latest update was 6 years ago) even with your PR. Should we close this? Can communicate to CRAN that we submitted a PR over a month ago.

@MichaelChirico
Copy link
Member

I think no harm in keeping it open while a CRAN release is pending, we can close once the release is official

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fread revdep Reverse dependencies WAI Working as intended
Projects
None yet
Development

No branches or pull requests

3 participants