Skip to content

Commit

Permalink
fix: error message for "too many arguments" (#7134
Browse files Browse the repository at this point in the history
Signed-off-by: Antonio Nuno Monteiro <[email protected]>
  • Loading branch information
anmonteiro authored Feb 21, 2023
1 parent b6d2872 commit 0c287fa
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion src/dune_sexp/decoder.ml
Original file line number Diff line number Diff line change
Expand Up @@ -230,7 +230,7 @@ let result : type a k. k context -> a * k -> a =
User_error.raise ~loc:(Ast.loc sexp) [ Pp.text "This value is unused" ]
| Some s ->
User_error.raise ~loc:(Ast.loc sexp)
[ Pp.textf "Too many argument for %s" s ]))
[ Pp.textf "Too many arguments for %s" s ]))
| Fields _ -> (
match Name.Map.choose state.unparsed with
| None -> v
Expand Down
2 changes: 1 addition & 1 deletion test/blackbox-tests/test-cases/alias-multiple.t
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,7 @@ We now update the dune file to use multiple aliases
File "dune", line 2, characters 10-11:
2 | (alias a b)
^
Error: Too many argument for alias
Error: Too many arguments for alias
[1]

That doesn't work so we use the aliases field
Expand Down

0 comments on commit 0c287fa

Please sign in to comment.