janestreet / janestreet/csvfields

Feature request: support additional parameters in csv load functions

Open
#2 0 comments 0 reactions 0 assignees View on GitHub
forwarded-to-js-devs
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

Open the contributing 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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.