bump dune version to 3, remove unused settings (#2817)

* bump dune version to 3, remove unused settings

* wip

Author: anmonteiro

dune-project

@@ -1,4 +1,4 @@
-(lang dune 2.9)
+(lang dune 3.8)
 
 (name reason)
 

reason.opam

@@ -15,7 +15,7 @@ license: "MIT"
 homepage: "https://reasonml.github.io/"
 bug-reports: "https://github.com/reasonml/reason/issues"
 depends: [
-  "dune" {>= "2.9"}
+  "dune" {>= "3.8"}
   "ocaml" {>= "4.06" & < "5.4"}
   "ocamlfind" {build}
   "dune-build-info" {>= "2.9.3"}
@@ -36,11 +36,9 @@ build: [
     name
     "-j"
     jobs
-    "--promote-install-files=false"
     "@install"
     "@runtest" {with-test}
     "@doc" {with-doc}
   ]
-  ["dune" "install" "-p" name "--create-install-files" name]
 ]
 dev-repo: "git+https://github.com/reasonml/reason.git"

rtop.opam

@@ -13,7 +13,7 @@ license: "MIT"
 homepage: "https://reasonml.github.io/"
 bug-reports: "https://github.com/reasonml/reason/issues"
 depends: [
-  "dune" {>= "2.9"}
+  "dune" {>= "3.8"}
   "ocaml" {>= "4.06" & < "5.4"}
   "reason" {= version}
   "utop" {>= "2.0"}
@@ -29,11 +29,9 @@ build: [
     name
     "-j"
     jobs
-    "--promote-install-files=false"
     "@install"
     "@runtest" {with-test}
     "@doc" {with-doc}
   ]
-  ["dune" "install" "-p" name "--create-install-files" name]
 ]
 dev-repo: "git+https://github.com/reasonml/reason.git"

src/menhir-recover/dune

@@ -1,3 +1,4 @@
 (executable
  (name main)
- (libraries fix menhirLib menhirSdk))
+ (libraries fix menhirLib menhirSdk)
+ (flags :standard -w -67))

src/reason-parser/dune

@@ -65,7 +65,7 @@
  (public_name reason)
  (wrapped false)
  (flags
-  (:standard -w -9-52 -safe-string))
+  (:standard -w -9-52-67 -safe-string))
  (modules
   ocaml_util
   reason_syntax_util

src/reason-parser/reason_pprint_ast.ml

@@ -794,17 +794,11 @@ type funcApplicationLabelStyle =
   | WrapFinalListyItemIfFewerThan of int
 
 type formatSettings =
-  { (* Whether or not to expect that the original parser that generated the AST
-       would have annotated constructor argument tuples with explicit arity to
-       indicate that they are multiple arguments. (True if parsed in original
-       OCaml AST, false if using Reason parser). *)
-    constructorTupleImplicitArity : bool
-  ; space : int
+  { space : int
   ; (* For curried arguments in function *definitions* only: Number of [space]s
        to offset beyond the [let] keyword. Default 1. *)
     listsRecordsIndent : int
   ; indentWrappedPatternArgs : int
-  ; indentMatchCases : int
   ; (* Amount to indent in label-like constructs such as wrapped function
        applications, etc - or even record fields. This is not the same concept
        as an indented curried argument list. *)
@@ -867,11 +861,9 @@ type formatSettings =
   }
 
 let defaultSettings =
-  { constructorTupleImplicitArity = false
-  ; space = 1
+  { space = 1
   ; listsRecordsIndent = 2
   ; indentWrappedPatternArgs = 2
-  ; indentMatchCases = 2
   ; indentAfterLabels = 2
   ; trySwitchIndent = 0
   ; funcApplicationLabelStyle = WrapFinalListyItemIfFewerThan 3