Opinionated OCaml linter with Merlin integration for code quality, naming conventions, and style checks
ocaml-sexpt: rename to ocaml-sexp
Drops the redundant 't' suffix that was a typo of 'sexp'. The package
keeps the same OCaml library name (Sexp), API, and tests; only the
on-disk directory name and opam package name change.
Updates:
- merlint/lib/dune, merlint/dune-project, merlint/merlint.opam:
swap the [sexpt] dependency for [sexp].
- merlint/lib/dune.ml: drop the [module Sexp = Sexpt.Sexp] alias
(the new package exports Sexp directly) and switch to the
result-typed [parse_string_many] now that the wrapper module is
gone.
Resolves the build error from earlier in the session where the
merlint/lib/dune still referenced the stale [sexpt] library.
+8
-6
+1
-1
dune-project
+1
-1
dune-project
+1
-1
lib/dune
+1
-1
lib/dune
+5
-3
lib/dune.ml
+5
-3
lib/dune.ml
···
1
1
(** Dune project structure analysis *)
2
2
3
-
module Sexp = Sexpt.Sexp
4
-
5
3
let src = Logs.Src.create "merlint.dune" ~doc:"Dune interface"
6
4
7
5
module Log = (val Logs.src_log src : Logs.LOG)
···
95
93
close_in ic;
96
94
97
95
(* Parse all S-expressions in the file *)
98
-
let stanzas = Sexp.parse_string_many_exn content in
96
+
let stanzas =
97
+
match Sexp.parse_string_many content with
98
+
| Ok s -> s
99
+
| Error e -> raise (Sexp.Error e)
100
+
in
99
101
Log.debug (fun m ->
100
102
m "Parsed dune file %a: found %d stanzas" Fpath.pp filename
101
103
(List.length stanzas));