Browse Source

Add missing v prefix to ppx_let lower bound

Kevin Ji 1 year ago
parent
commit
3a070d6527
3 changed files with 1 additions and 6 deletions
  1. 0 1
      .travis.yml
  2. 1 1
      angstrom.opam
  3. 0 4
      lib_test/test_let_syntax_ppx.ml

+ 0 - 1
.travis.yml

@@ -10,7 +10,6 @@ env:
   - PINS="angstrom-async:. angstrom-lwt-unix:. angstrom:."
   - PACKAGE="angstrom"
   - TESTS=true
-  - EXTRA_DEPS="ppx_let"
   - POST_INSTALL_HOOK="opam install --with-test angstrom-async angstrom-lwt-unix && opam exec -- make examples"
   matrix:
   - OCAML_VERSION="4.08"

+ 1 - 1
angstrom.opam

@@ -15,7 +15,7 @@ depends: [
   "dune" {>= "1.8"}
   "alcotest" {with-test & >= "0.8.1"}
   "bigstringaf"
-  "ppx_let" {with-test & >= "0.14.0"}
+  "ppx_let" {with-test & >= "v0.14.0"}
   "ocaml-syntax-shims" {build}
 ]
 synopsis: "Parser combinators built for speed and memory-efficiency"

+ 0 - 4
lib_test/test_let_syntax_ppx.ml

@@ -11,12 +11,8 @@ let (_ : int t) =
   in
   2
 
-(* [mapn] support was introduced in ppx_let.v0.14.0, which CI does not reliably
-  install. *)
-(*
 let (_ : int t) =
   let%mapn (_ : char) = any_char
   and (_ : string) = string "foo"
   in
   2
-*)