Skip to content

Commit

Permalink
fix: remove warning 63-70 by default
Browse files Browse the repository at this point in the history
In particular, warning 70 is very annoying

Signed-off-by: Rudi Grinberg <me@rgrinberg.com>

ps-id: F37A3512-7207-43AC-846F-EFE7070F82CB
  • Loading branch information
rgrinberg committed Feb 24, 2022
1 parent 11b6c5f commit 24ad689
Showing 1 changed file with 23 additions and 2 deletions.
25 changes: 23 additions & 2 deletions src/dune_rules/ocaml_flags.ml
Original file line number Diff line number Diff line change
Expand Up @@ -11,8 +11,29 @@ let dev_mode_warnings =
(* New warnings should be introduced here *)
let all =
Int.Set.diff
(Int.Set.of_list (List.init 62 ~f:succ))
(Int.Set.of_list [ 4; 29; 40; 41; 42; 44; 45; 48; 58; 59; 60 ])
(* TODO update this list once OCaml versions are out *)
(Int.Set.of_list (List.init 70 ~f:succ))
(Int.Set.of_list
[ 4
; 29
; 40
; 41
; 42
; 44
; 45
; 48
; 58
; 59
; 60
; 63
; 64
; 65
; 66
; 67
; 68
; 69
; 70
])
in
let warnings_range ws =
let wrange_to_flag (x, y) =
Expand Down

0 comments on commit 24ad689

Please sign in to comment.