Skip to content

Commit

Permalink
test: Remove 'type_desc_to_yojson' library
Browse files Browse the repository at this point in the history
The library is installable and is built when the tests are not going to
be run.
This fails in the lower-bounds CI because of the requirement on yojson
is not respected when the dependencies are installed with 'with-test'.

    "yojson" {>= "2.1.0" & with-test}
  • Loading branch information
Julow authored and jonludlam committed Jan 24, 2025
1 parent 8568008 commit 5eaf6fa
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 17 deletions.
5 changes: 4 additions & 1 deletion test/model/semantics/dune
Original file line number Diff line number Diff line change
@@ -1,3 +1,6 @@
(rule
(copy ../../odoc_print/type_desc_to_yojson.ml type_desc_to_yojson.ml))

(library
(name odoc_model_semantics_test)
(package odoc)
Expand All @@ -7,4 +10,4 @@
(>= %{ocaml_version} 4.04.1))
(preprocess
(pps ppx_expect))
(libraries sexplib0 odoc_model type_desc_to_yojson))
(libraries sexplib0 odoc_model odoc_model_desc yojson))
19 changes: 3 additions & 16 deletions test/odoc_print/dune
Original file line number Diff line number Diff line change
@@ -1,25 +1,12 @@
(library
(name type_desc_to_yojson)
(package odoc)
(optional)
(modules type_desc_to_yojson)
(libraries odoc_model_desc yojson))

(executable
(name odoc_print)
(modules odoc_print)
(libraries odoc_odoc type_desc_to_yojson odoc_model_desc compatcmdliner))
(modules odoc_print type_desc_to_yojson)
(libraries odoc_odoc odoc_model_desc yojson compatcmdliner))

(executable
(name print_index)
(modules print_index)
(libraries
odoc_odoc
type_desc_to_yojson
odoc_model_desc
compatcmdliner
yojson
odoc_json_index))
(libraries odoc_odoc odoc_model_desc compatcmdliner yojson odoc_json_index))

(executable
(name occurrences_print)
Expand Down

0 comments on commit 5eaf6fa

Please sign in to comment.