Skip to content

Switch to dune.3

Hugo Heuzard requested to merge nomadic-labs/tezos:hhugo-move-to-dune3 into master

Context

After !5257 (merged) Replaces !4099 (closed), !5411 (merged) Goes with opam-repository!294 (merged)

Fix #2009 (closed), #2008 (closed), #1947 (closed)

  • remove some deadcode. Dune 3 generate empty mli for executables and tests allowing the compiler to complain about unused values.
  • plug some tests, there were splotted as deadcode by the compiler
  • remove our hack for running inline_tests in javascript. Dune 3 now allows to specify a custom alias for thoses.
  • drop the ppx_blob dependency (it doesn't work welll with dune 3), replaced by ocp-ocamlres that we use else where in the codebase.

Manually testing the MR

Checklist

  • Document the interface of any function added or modified (see the coding guidelines)
  • Document any change to the user interface, including configuration parameters (see node configuration)
  • Provide automatic testing (see the testing guide).
  • For new features and bug fixes, add an item in the appropriate changelog (docs/protocols/alpha.rst for the protocol and the environment, CHANGES.rst at the root of the repository for everything else).
  • Select suitable reviewers using the Reviewers field below.
  • Select as Assignee the next person who should take action on that MR
Edited by Hugo Heuzard

Merge request reports