Skip to content

Commit 175296c

Browse files
committed
Merge branch 'fix-unused-libs'
2 parents 6d46ef1 + 868177f commit 175296c

File tree

6 files changed

+27
-6
lines changed

6 files changed

+27
-6
lines changed

cmdlang-tests.opam

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,6 @@ depends: [
2121
"core" {>= "v0.17"}
2222
"core_unix" {>= "v0.17"}
2323
"expect_test_helpers_core" {>= "v0.17"}
24-
"loc" {>= "0.2.2"}
2524
"mdx" {>= "2.4"}
2625
"ppx_compare" {>= "v0.17"}
2726
"ppx_enumerate" {>= "v0.17"}

doc/docs/guides/usage-styles/lib/dune

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,13 @@
44
(flags :standard -w +a-4-40-41-42-44-45-48-66 -warn-error +a -open Cmdlang)
55
(libraries cmdlang)
66
(lint
7-
(pps ppx_js_style -allow-let-operators -check-doc-comments))
7+
(pps
8+
ppx_js_style
9+
-allow-let-operators
10+
-check-doc-comments
11+
;; A workaround for an issue with this particular lint in that repo.
12+
;; See also [https://github.com/janestreet/ppx_js_style/issues/4].
13+
-allow-unannotated-ignores))
814
(preprocess
915
(pps
1016
-unused-code-warnings=force

dune-project

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -124,8 +124,6 @@
124124
(>= v0.17))
125125
(expect_test_helpers_core
126126
(>= v0.17))
127-
(loc
128-
(>= 0.2.2))
129127
(mdx
130128
(>= 2.4))
131129
(ppx_compare

test/cmdlang/ast/test__ast.ml

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,14 @@
1+
(*********************************************************************************)
2+
(* cmdlang - Declarative command-line parsing for OCaml *)
3+
(* SPDX-FileCopyrightText: 2024-2025 Mathieu Barbin <mathieu.barbin@gmail.com> *)
4+
(* SPDX-License-Identifier: MIT *)
5+
(*********************************************************************************)
6+
7+
(* This is to silence `dune build @unused-libs` and keeping intended deps. *)
8+
open! Cmdlang_ast.Ast
9+
10+
let%expect_test "empty" =
11+
();
12+
[%expect {||}];
13+
()
14+
;;

test/cmdlang/ast/test__ast.mli

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
(*_********************************************************************************)
2+
(*_ cmdlang - Declarative command-line parsing for OCaml *)
3+
(*_ SPDX-FileCopyrightText: 2024-2025 Mathieu Barbin <mathieu.barbin@gmail.com> *)
4+
(*_ SPDX-License-Identifier: MIT *)
5+
(*_********************************************************************************)

test/expect/dune

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -23,8 +23,7 @@
2323
cmdliner
2424
core.command
2525
core_unix.command_unix
26-
expect_test_helpers_core.expect_test_helpers_base
27-
loc)
26+
expect_test_helpers_core.expect_test_helpers_base)
2827
(instrumentation
2928
(backend bisect_ppx))
3029
(lint

0 commit comments

Comments
 (0)