@@ -44,55 +44,55 @@ let encode { name; constraint_ } =
44
44
(* Check for common typos in package dependency constraints *)
45
45
let check_for_typo ~loc { name; constraint_ } =
46
46
(* Handle case where config system isn't initialized yet *)
47
- let warnings_enabled =
48
- try
49
- Dune_config.Config. (get typo_warnings) = `Enabled
50
- with
51
- | _ -> true (* Default to enabled when config unavailable *)
47
+ let warnings_enabled =
48
+ try Dune_config.Config. (get typo_warnings) = `Enabled with
49
+ | _ -> true (* Default to enabled when config unavailable *)
52
50
in
53
51
(* Early return if warnings are disabled *)
54
- if not warnings_enabled then None
55
- else
56
- (match constraint_ with
57
- | Some
58
- (Package_constraint. Uop
59
- (Relop. Eq , Package_constraint.Value. String_literal " version" )) ->
60
- let message =
61
- User_message. make
62
- ~loc
63
- [ Pp. textf
64
- " Possible typo in constraint for dependency %S: '(= version)' might be a \
65
- mistake."
66
- (Package_name. to_string name)
67
- ]
68
- ~hints:
69
- [ Pp. textf
70
- " Did you mean to use the `:version` variable instead? Example: (depends \
71
- (%s (= :version)))"
72
- (Package_name. to_string name)
73
- ]
74
- in
75
- Some message
76
- | Some (Package_constraint. Bvar var)
77
- when String. equal (Package_variable_name. to_string var) " with_test" ->
78
- let message =
79
- User_message. make
80
- ~loc
81
- [ Pp. textf
82
- " Possible typo in constraint for dependency %S: ':with_test' might be a \
83
- mistake."
84
- (Package_name. to_string name)
85
- ]
86
- ~hints:
87
- [ Pp. textf
88
- " Did you mean to use ':with-test' instead? Example: (depends (%s \
89
- :with-test))"
90
- (Package_name. to_string name)
91
- ]
92
- in
93
- Some message
94
- | _ -> None )
95
-
52
+ if not warnings_enabled
53
+ then None
54
+ else (
55
+ match constraint_ with
56
+ | Some
57
+ (Package_constraint. Uop
58
+ (Relop. Eq , Package_constraint.Value. String_literal " version" )) ->
59
+ let message =
60
+ User_message. make
61
+ ~loc
62
+ [ Pp. textf
63
+ " Possible typo in constraint for dependency %S: '(= version)' might be a \
64
+ mistake."
65
+ (Package_name. to_string name)
66
+ ]
67
+ ~hints:
68
+ [ Pp. textf
69
+ " Did you mean to use the `:version` variable instead? Example: (depends \
70
+ (%s (= :version)))"
71
+ (Package_name. to_string name)
72
+ ]
73
+ in
74
+ Some message
75
+ | Some (Package_constraint. Bvar var)
76
+ when String. equal (Package_variable_name. to_string var) " with_test" ->
77
+ let message =
78
+ User_message. make
79
+ ~loc
80
+ [ Pp. textf
81
+ " Possible typo in constraint for dependency %S: ':with_test' might be a \
82
+ mistake."
83
+ (Package_name. to_string name)
84
+ ]
85
+ ~hints:
86
+ [ Pp. textf
87
+ " Did you mean to use ':with-test' instead? Example: (depends (%s \
88
+ :with-test))"
89
+ (Package_name. to_string name)
90
+ ]
91
+ in
92
+ Some message
93
+ | _ -> None )
94
+ ;;
95
+
96
96
let decode =
97
97
let open Dune_sexp.Decoder in
98
98
let constrained =
0 commit comments