Skip to content
This repository has been archived by the owner on May 5, 2019. It is now read-only.

deprecate readtable & writetable for CSV.read & CSV.write #10

Closed
4 tasks
cjprybol opened this issue Feb 15, 2017 · 5 comments
Closed
4 tasks

deprecate readtable & writetable for CSV.read & CSV.write #10

cjprybol opened this issue Feb 15, 2017 · 5 comments

Comments

@cjprybol
Copy link
Contributor

cjprybol commented Feb 15, 2017

a find & replace of DataFrame -> DataTable project-wide for each was sufficient to get those working.

after those changes we'll deprecate

  • readtable in favor of CSV.read
  • writetable in favor of CSV.write

I've seen this discussed but couldn't find an open issue

@nalimilan
Copy link
Member

Cc: @quinnj

@deyandyankov
Copy link

I just want to point out that writetable and readtable are still in the online docs but are not defined in the module (or module docs). Perhaps the online documentation needs to be rebuilt?

@nalimilan
Copy link
Member

Indeed, there's an error when building the manual: ERROR: LoadError: 'man/io.md' is not an existing page!. That reference should be removed from the files.

@nalimilan
Copy link
Member

See #71.

@cjprybol cjprybol changed the title Open DataTables branches on DataStreams.jl & CSV.jl deprecate readtable & writetable for CSV.read & CSV.write Aug 6, 2017
@cjprybol
Copy link
Contributor Author

cjprybol commented Aug 6, 2017

#26

@cjprybol cjprybol closed this as completed Aug 6, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants