New release of datakit

[samoht]

CHANGES:

• all: update to latest version of alcotest, conduit, session, ocaml-github,
  ocaml-github-hooks and cohttp (attempt to fix CI moby/datakit#612, @samoht and @djs55)

• github: make User.t abstract (github: make user an abstract type moby/datakit#594, @samoht)

• github: turn Webhook.events into a promise (github: turn Webhook.events into a promise moby/datakit#598, @samoht)

• github: add a Comment module to model PR and issue comments (sync PR's comments moby/datakit#595, @samoht)

• github: change PR.owner to be of type User.t (github: make PR.owner a "user" type moby/datakit#599, @samoht)

• github-bridge: add the ability to sync PR's coments (sync PR's comments moby/datakit#595, @samoht)

• go-client: handle large values when reading / writing in 9db (Go-datakit handle large values when reading / writing in 9db moby/datakit#292, @simonferquel)

• go-client: fix the handling of defaults over upgrade (Go: fix the handling of defaults over upgrade moby/datakit#605, @djs55)

• go-client: improve transaction API (Go: improve transaction API moby/datakit#606, @djs55)

[samoht]

/cc @djs55 and @talex5

[camelus]

✅ All lint checks passed cff633d

• These packages passed lint tests: datakit-bridge-github.0.12.0, datakit-bridge-local-git.0.12.0, datakit-ci.0.12.0, datakit-client-9p.0.12.0, datakit-client-git.0.12.0, datakit-client.0.12.0, datakit-github.0.12.0, datakit-server-9p.0.12.0, datakit-server.0.12.0, datakit.0.12.0

---

✅ Installability check (7790 → 7800)

• new installable packages (10): datakit.0.12.0 datakit-bridge-github.0.12.0
datakit-bridge-local-git.0.12.0 datakit-ci.0.12.0 datakit-client.0.12.0
datakit-client-9p.0.12.0 datakit-client-git.0.12.0
datakit-github.0.12.0 datakit-server.0.12.0 datakit-server-9p.0.12.0

[samoht]

cohttp.0.99 failure on 4.03:

[31m#===�[m �[01mERROR�[m �[01mwhile installing cohttp.0.99.0�[m �[31m=====================================#�[m
�[33m#�[m �[33mopam-version�[m 1.2.2
�[33m#�[m �[33mos�[m           linux
�[33m#�[m �[33mcommand�[m      jbuilder build -p cohttp -j 4
�[33m#�[m �[33mpath�[m         /home/travis/.opam/4.03.0/build/cohttp.0.99.0
�[33m#�[m �[33mcompiler�[m     4.03.0
�[33m#�[m �[33mexit-code�[m    1
�[33m#�[m �[33menv-file�[m     /home/travis/.opam/4.03.0/build/cohttp.0.99.0/cohttp-30047-ab0078.env
�[33m#�[m �[33mstdout-file�[m  /home/travis/.opam/4.03.0/build/cohttp.0.99.0/cohttp-30047-ab0078.out
�[33m#�[m �[33mstderr-file�[m  /home/travis/.opam/4.03.0/build/cohttp.0.99.0/cohttp-30047-ab0078.err
�[33m### stderr ###
�[m�[33m# �[m[...]
�[33m# �[mFile "cohttp/src/link.ml", line 123, characters 10-11:
�[33m# �[mError: Multiple definition of the module name Fields.
�[33m# �[m       Names must be unique in a given structure or signature.
�[33m# �[m    ocamlopt cohttp/src/cohttp__Link.{cmx,o} (exit 2)
�[33m# �[m(cd _build/default && /home/travis/.opam/4.03.0/bin/ocamlopt.opt -w -40 -g -I /home/travis/.opam/4.03.0/lib/base -I /home/travis/.opam/4.03.0/lib/base/caml -I /home/travis/.opam/4.03.0/lib/base/shadow_stdlib -I /home/travis/.opam/4.03.0/lib/base64 -I /home/travis/.opam/4.03.0/lib/bytes -I /home/travis/.opam/4.03.0/lib/fieldslib -I /home/travis/.opam/4.03.0/lib/ocaml -I /home/travis/.opam/4.03.0/lib/ppx_deriving -I /home/travis/.opam/4.03.0/lib/re -I /home/travis/.opam/4.03.0/lib/result -I /home/travis/.opam/4.03.0/lib/sexplib -I /home/travis/.opam/4.03.0/lib/sexplib/0 -I /home/travis/.opam/4.03.0/lib/stringext -I /home/travis/.opam/4.03.0/lib/uri -no-alias-deps -I cohttp/src -open Cohttp -o cohttp/src/cohttp__Link.cmx -c -impl cohttp/src/link.pp.ml)
�[33m# �[mFile "cohttp/src/link.ml", line 123, characters 10-11:
�[33m# �[mError: Multiple definition of the module name Fields.
�[33m# �[m       Names must be unique in a given structure or signature.
�[33m# �[m
�[33m# �[mWaiting for 1 job to finish.

Note sure what this means ...
All the datakit-ci issues are due to the constraint solver timing out.

[samoht]

Ha yes, the issue is ocaml-ppx/ppx_deriving#147

So I guess cohttp needs a lower bound on the ppx_deriving to use.

[hannesm]

@samoht shouldn't instead the ppx_deriving versions which generate duplicated code have an available != 4.03.0? or is the issue only with deriving and cohttp?

[samoht]

@hannesm that's probably the right fix indeed!

[samoht]

I guess that's the same issue as #10807

[samoht]

Rebased on top of master to get the fix from #10807

[avsm]

Thanks! You may want to announce this on https://discuss.ocaml.org, where we have a Community category and an announce tag for this purpose.