docs: remove warnings, part 1

jscomp/core/lam.mli

@@ -135,10 +135,10 @@ val false_ : t
 val unit : t
 
 val sequor : t -> t -> t
-(** convert [l || r] to [if l then true else r]*)
+(** convert [l || r] to [if l then true else r] *)
 
 val sequand : t -> t -> t
-(** convert [l && r] to [if l then r else false *)
+(** convert [l && r] to [if l then r else false] *)
 
 val not_ : Location.t -> t -> t
 (** constant folding *)

jscomp/melstd/array.mli

@@ -27,7 +27,7 @@ include module type of struct
 end
 
 val reverse_range : 'a array -> int -> int -> unit
-(** Some utilities for {!Array} operations *)
+(** Some utilities for array operations *)
 
 val reverse_of_list : 'a list -> 'a array
 val to_list_f : 'a array -> ('a -> 'b) -> 'b list

jscomp/melstd/hash_set.mli

@@ -22,8 +22,8 @@
  * along with this program; if not, write to the Free Software
  * Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA. *)
 
-(** Ideas are based on {!Hash}, 
-    however, {!Hash.add} does not really optimize and has a bad semantics for {!Hash_set}, 
+(** Ideas are based on [Hash], 
+    however, [Hash.add] does not really optimize and has a bad semantics for [Hash_set], 
     This module fixes the semantics of [add].
     [remove] is not optimized since it is not used too much 
 *)

jscomp/others/belt_Id.mli

@@ -66,7 +66,7 @@ type ('key, 'id) comparable =
     Unlike normal functions, when created, it comes with a unique identity (guaranteed
     by the type system).
 
-    It can be created using function {!comparableU} or{!comparable}.
+    It can be created using function {!comparableU} or{!val-comparable}.
 
     The idea of a unique identity when created is that it makes sure two sets would type
     mismatch if they use different comparison function
@@ -117,7 +117,7 @@ type ('key, 'id) hashable =
     Unlike normal functions, when created, it comes with a unique identity (guaranteed
     by the type system).
 
-    It can be created using function {!hashableU} or {!hashable}.
+    It can be created using function {!hashableU} or {!val-hashable}.
 
     The idea of a unique identity when created is that it makes sure two hash sets would type
     mismatch if they use different comparison function

jscomp/others/belt_Range.mli

@@ -47,7 +47,7 @@ val everyByU : int -> int -> step:int -> ((int -> bool)[@u]) -> bool
 val everyBy : int -> int -> step:int -> (int -> bool) -> bool
 (** [everyBy start finish ~step p]
 
-    {b See} {!Belt_Array.rangeBy}
+    {b See} {!Belt.Array.rangeBy}
 
     equivalent to [Belt.Array.(every (rangeBy start finish ~step) p)]
 *)
@@ -65,7 +65,7 @@ val someByU : int -> int -> step:int -> ((int -> bool)[@u]) -> bool
 val someBy : int -> int -> step:int -> (int -> bool) -> bool
 (** [someBy start finish ~step  p]
 
-    {b See} {!Belt_Array.rangeBy}
+    {b See} {!Belt.Array.rangeBy}
 
     equivalent to [Belt.Array.(some (rangeBy start finish ~step) p)]
 *)

jscomp/others/belt_Set.cppo.mli

@@ -23,7 +23,7 @@
  * Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA. *)
 
 (** This module is {!Belt.Set} specialized with value type to be a primitive type.
-    It is more efficient in general, the  API is the same with {!Belt_Set} except its value type is fixed,
+    It is more efficient in general, the  API is the same with {!Belt.Set} except its value type is fixed,
     and identity is not needed(using the built-in one)
 
     {b See} {!Belt.Set}

jscomp/others/sort.cppo.mli

@@ -24,7 +24,7 @@
  * Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA. *)
 
 
-(** This is a specialized module for {!Belt_SortArray}, the docs in that module also
+(** This is a specialized module for {!Belt.SortArray}, the docs in that module also
     applies here, except the comparator is fixed and inlined
 *)
 
@@ -38,7 +38,7 @@ type element = string
 
 val strictlySortedLength: element array -> int
 (**
-  The same as {!Belt_SortArray.strictlySortedLength } except the comparator is fixed
+  The same as {!Belt.SortArray.strictlySortedLength } except the comparator is fixed
   @return [+n] means increasing order  [-n] means negative order
 *)
 
@@ -48,11 +48,11 @@ val isSorted: element array  -> bool
 
 val stableSortInPlace: element array -> unit
 (**
-   The same as {!Belt_SortArray.stableSortInPlaceBy} except the comparator is fixed
+   The same as {!Belt.SortArray.stableSortInPlaceBy} except the comparator is fixed
 *)
 
 val stableSort: element array -> element array
-(** The same as {!Belt_SortArray.stableSortBy} except the comparator is fixed *)
+(** The same as {!Belt.SortArray.stableSortBy} except the comparator is fixed *)
 
 val binarySearch: element array -> element -> int
 (**

jscomp/runtime/js.ml

@@ -73,7 +73,7 @@ module Null = Js_null
 (** Provide utilities around ['a null] *)
 
 module Undefined = Js_undefined
-(** Provide utilities around {!undefined} *)
+(** Provide utilities around {!type-undefined} *)
 
 module Nullable = Js_nullable
 (** Provide utilities around {!null_undefined} *)

jscomp/runtime/js_internal.ml

@@ -88,7 +88,7 @@ external nullToOption : 'a null -> 'a option = "#null_to_opt"
 external isNullable : 'a nullable -> bool = "#is_nullable"
 
 external testAny : 'a -> bool = "#is_nullable"
-(** The same as {!test} except that it is more permissive on the types of input *)
+(** The same as {!isNullable} except that it is more permissive on the types of input *)
 
 external null : 'a null = "#null"
 (** The same as [empty] in {!Js.Null} will be compiled as [null]*)

jscomp/runtime/js_json.mli

@@ -174,6 +174,7 @@ let getIds s =
   (* prints `1, 2, 3` *)
   let _ =
     Js.log (getIds {| { "ids" : [1, 2, 3] } |})
+]}
 
 @see <https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/JSON/parse> MDN
 *)

jscomp/runtime/js_promise.ml

@@ -24,9 +24,6 @@
 
 (** Specialized bindings to Promise. Note: For simplicity,
     this binding does not track the error type, it treat it as an opaque type
-    {[
-
-    ]}
 *)
 
 type +'a t

jscomp/runtime/js_re.ml

@@ -36,13 +36,9 @@ and subsequent uses will ocntinue the search from the previous {! lastIndex}.
 let maybeMatches = "banana" |> Js.String.match_ [\[%re "/na+/g"\]]
 ]}
 
-@see
-  <https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/RegExp>
-  JavaScript API reference on MDN
+@see <https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/RegExp> JavaScript API reference on MDN
 
-@see
-  <https://developer.mozilla.org/en-US/docs/Web/JavaScript/Guide/Regular_Expressions>
-  JavaScript Regular Expressions Guide on MDN
+@see <https://developer.mozilla.org/en-US/docs/Web/JavaScript/Guide/Regular_Expressions> JavaScript Regular Expressions Guide on MDN
 *)
 
 type t

jscomp/runtime/js_string.ml

@@ -68,7 +68,6 @@ external fromCodePoint : int -> t = "String.fromCodePoint"
   fromCodePoint 0xd55c = {js|한|js};;
   fromCodePoint 0x1f63a = {js|😺|js};;
 ]}
-
 *)
 
 external fromCodePointMany : int array -> t = "String.fromCodePoint"
@@ -87,7 +86,6 @@ external length : t -> int = "length"
 {[
   length "abcd" = 4;;
 ]}
-
 *)
 
 external get : t -> int -> t = ""

jscomp/runtime/js_undefined.mli

@@ -34,8 +34,8 @@ external return : 'a -> 'a t = "%identity"
 
 val testAny : 'a -> bool
 (**
-   @since 1.6.1
    Returns [true] if the given value is [empty] ([undefined])
+   @since 1.6.1
 *)
 
 external empty : 'a t = "#undefined"