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

Doc: CSV.jl #44909

Closed
wants to merge 3 commits into from
Closed

Doc: CSV.jl #44909

wants to merge 3 commits into from

Conversation

PallHaraldsson
Copy link
Contributor

@PallHaraldsson PallHaraldsson commented Apr 8, 2022

I tried to search the docs for CSV and found lots of false alarms, e.g. the C standard library, and nothing of value, while mentioned indirecty in e.g.
https://github.com/JuliaLang/julia/blob/020c2de5bd1198c97782615c4a3572706d0f4977/doc/src/manual/modules.md

"Fix" or at least workaround for JuliaData/DelimitedFiles.jl#1 where JuliaData/DelimitedFiles.jl#1 claimed (while I can't see it clearly officially stated): "readdlm is also effectively deprecated and the CSV package should be used."

I tried to search the docs for CSV and found lots of false alarms, e.g. the C standard library, and nothing of value, while mentioned indirecty in e.g.
https://github.com/JuliaLang/julia/blob/020c2de5bd1198c97782615c4a3572706d0f4977/doc/src/manual/modules.md
@ViralBShah ViralBShah added the docs This change adds or pertains to documentation label Apr 8, 2022
@ViralBShah
Copy link
Member

The plan is to move DelimitedFiles.jl out of stdlib into a standalone package, but we need someone to help make it happen. #44663

@PallHaraldsson
Copy link
Contributor Author

Good to know, can't this still be merged in the meantime (to 1.8 since just docs)? Right now, or eventually add backport to 1.6 (1.7 not need). I'm going on a holiday for a week, so I will not be editing the PR. And I forgot to add [skip ci], as would have been wise for docs... I guess pending checks can be stopped.

@ViralBShah
Copy link
Member

It can be merged. It just needs some wordsmithing. I will get to it over the weekend.

@ViralBShah ViralBShah closed this Apr 9, 2022
@ViralBShah
Copy link
Member

The package is already separated out. So we need to make this change in the new place. This doc update is still valuable for 1.8.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs This change adds or pertains to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants