Skip to content
This repository was archived by the owner on Jun 15, 2023. It is now read-only.

Remove references to reasonreact in react ppx diagnostics #458

Merged
merged 1 commit into from
Aug 30, 2021
Merged

Remove references to reasonreact in react ppx diagnostics #458

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 3 additions & 3 deletions src/reactjs_jsx_ppx_v3.ml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -361,7 +361,7 @@ let jsxMapper () =
Location.prerr_warning pattern.ppat_loc
(Preprocessor
(Printf.sprintf
"ReasonReact: optional argument annotations must have explicit `option`. Did you mean \
"React: optional argument annotations must have explicit `option`. Did you mean \
`option(%s)=?`?"
currentType)) )
| _ -> ()
Expand All @@ -386,7 +386,7 @@ let jsxMapper () =
(list, Some txt)
| Pexp_fun (Nolabel, _, pattern, _expression) ->
Location.raise_errorf ~loc:pattern.ppat_loc
"ReasonReact: react.component refs only support plain arguments and type annotations."
"React: react.component refs only support plain arguments and type annotations."
| _ -> (list, None)
[@@raises Invalid_argument]
in
Expand Down Expand Up @@ -562,7 +562,7 @@ let jsxMapper () =
if hasApplication.contents then ((fun a -> a), false, unerasableIgnoreExp expression)
else
Location.raise_errorf ~loc:pattern.ppat_loc
"ReasonReact: props need to be labelled arguments.\n\
"React: props need to be labelled arguments.\n\
\ If you are working with refs be sure to wrap with React.forwardRef.\n\
\ If your component doesn't have any props use () or _ instead of a name."
(* let make = {let foo = bar in (~prop) => ...} *)
Expand Down