Conversation
Member
|
@patricoferris @lyrm @art-w @ElectreAAS @clecat -- are you happy to merge this in |
Contributor
Author
|
I do not see any major blockers |
|
I made a list of things that needed fixes here . This is clearly outdated, but it's likely that not everything has been done. A lot of it is small tasks (updating examples and documentation), and it was decided that it would be done before merging to main, if I remembered correctly. |
Contributor
|
I think #2318 should also be included |
Fix array shuffle
Fix libirmin build
libirmin: reuse a single eio scheduler across calls
Fix repo initialization with opam
* Qcheck stm test for irmin * Add irmin-pack tests * Some debugging. * Fix dune file. * Replace deprecated functions * Update pin to qcheck-stm
* Add '/' to allowed branch characters Signed-off-by: Benjamin Somers <benjamin.somers@imt-atlantique.fr> * Add funding manifest discovery See https://floss.fund/funding-manifest/ * Update manifest URL --------- Signed-off-by: Benjamin Somers <benjamin.somers@imt-atlantique.fr> Co-authored-by: Benjamin Somers <benjamin.somers@imt-atlantique.fr> Co-authored-by: Thomas Gazagnaire <thomas@gazagnaire.org>
* Fix test_stm by using posix Eio backend The test_stm test was disabled because it exhausted io_uring resources on Linux (ENOMEM on io_uring_queue_init when spawning multiple Eio domains). Instead of disabling the test, use the posix backend via EIO_BACKEND environment variable. This requires converting the test stanza to an executable + rule pattern to allow setting the environment variable.
* Fix formatting in test logs Signed-off-by: Ambre Austen Suhamy <ambre@tarides.com> * Update ocamlformat 0.28.1 * ppx_irmin: use ppxlib >= 0.37 * pin bisect_ppx --------- Signed-off-by: Ambre Austen Suhamy <ambre@tarides.com> Co-authored-by: Arthur Wendling <arthur@tarides.com> Co-authored-by: zach <zachshipko@gmail.com>
* Fix irmin-watcher hash to newest Signed-off-by: Ambre Austen Suhamy <ambre@tarides.com> * Added setup-dune CI workflow Signed-off-by: Ambre Austen Suhamy <ambre@tarides.com> * fix formatting --------- Signed-off-by: Ambre Austen Suhamy <ambre@tarides.com> Co-authored-by: zach <zachshipko@gmail.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
8 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This pull requests goal is to finalize to move from lwt to eio within the irmin project, it should be merged and reviewed once we are satisfied by the state of the eio branch and most of the PRs pointing to it are merged.
It will also allow to keep an eye on the status of the CI.