janestreet / janestreet/csvfields
Feature request: support additional parameters in csv load functions
- Dominant language
- OCaml
- Stars
- 9
- Forks
- 3
- PR merge metrics
- No merged PRs in 30d
Description
Newer versions of `ocaml-csv` support useful configuration options such as `has_header`, `strip`, and `fix`. Although I think `ocaml-csv` was re-licensed from LGPLv1 to LGPLv2 since being included in this library's source code, the static linking exemption is still present, so it should still be possible to publish this library under MIT (disclaimer: I am not a lawyer).
I assume based on https://github.com/janestreet/csvfields/tree/master/ocaml-csv that the plan was to replace `ocaml-csv` with a different library called `Delimited`, but I haven't been able to find that anywhere. Are there any plans to release this change, or to update the version of `ocaml-csv` to support the additional parameters?
Contributor guide
Research direction
Start by locating the csv load functions and comparing their current API with the newer ocaml-csv options named in the issue: has_header, strip, and fix. Investigate the referenced Delimited project and the repository's dependency and licensing context. Done means the project has a decided implementation path and the requested configuration options are supported or the release plan is documented.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- ocaml
- Domain
- data
- Issue type
- Feature
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 25/100