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

Package csv-lwt.2.0 #10215

Closed
wants to merge 1 commit into from
Closed

Conversation

Chris00
Copy link
Member

@Chris00 Chris00 commented Sep 4, 2017

csv-lwt.2.0

A pure OCaml library to read and write CSV files, LWT version.

This is a pure OCaml library to read and write CSV files, including
all extensions used by Excel — e.g. quotes, newlines, 8 bit characters
in fields, "0 etc. A special representation of rows of CSV files with
a header is provided. The library comes with a handy command line tool
called csvtool for handling CSV files from shell scripts. This
version can be used with the monadic concurrency library LWT.




2.0 2017-09-02

  • Split the package into csv and csv-lwt.
  • Port to jbuilder and topkg.
    🐫 Pull-request generated by opam-publish v0.3.5

@camelus
Copy link
Contributor

camelus commented Sep 4, 2017

✅ All lint checks passed c5a0463
  • These packages passed lint tests: csv-lwt.2.0

❗ Installability check (7358 → 7358)
  • new broken packages (1): csv-lwt.2.0

@Chris00 Chris00 force-pushed the opam-publish-csv-lwt.2.0 branch from 6b7c16e to b1c2257 Compare September 4, 2017 02:19
@Chris00
Copy link
Member Author

Chris00 commented Sep 4, 2017

When releasing multiple related packages, are we supposed to put a version constraint to keep them in sync? If so, a {= version} (where version is a variable for the current version of the package) would be useful.

@Chris00 Chris00 force-pushed the opam-publish-csv-lwt.2.0 branch 2 times, most recently from aa9ad50 to b32513d Compare September 4, 2017 11:40
@Chris00 Chris00 force-pushed the opam-publish-csv-lwt.2.0 branch from b32513d to c5a0463 Compare September 4, 2017 11:44
@samoht
Copy link
Member

samoht commented Sep 5, 2017

Can you rebase with master now that I have merged csv.2.0.? You could also submit both packages in one PR, this would have been testing the two packages.

@Chris00
Copy link
Member Author

Chris00 commented Sep 6, 2017

You could also submit both packages in one PR

@dbuenzli Is it possible with topkg? Passing several -n at once?

@Chris00 Chris00 closed this Sep 6, 2017
@Chris00 Chris00 deleted the opam-publish-csv-lwt.2.0 branch September 6, 2017 06:53
@Chris00
Copy link
Member Author

Chris00 commented Sep 6, 2017

See #10226

@samoht
Copy link
Member

samoht commented Sep 6, 2017

It is not possible to do it yet because of ocaml-opam/opam-publish#38

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants