Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: fix some comments #2782

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
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
8 changes: 4 additions & 4 deletions src/menhir-recover/emitter.ml
Original file line number Diff line number Diff line change
Expand Up @@ -36,10 +36,10 @@ end = struct
| (Abort | Reduce _ | Shift _) as a -> a
| Seq [ v ] -> normalize_action v
| Seq v -> (match normalize_actions v with [ x ] -> x | xs -> Seq xs)

(* Find sharing opportunities. If the same sequence of action occurs multiple
times, the funtion will associate a unique identifier to the sequence.

(* Find sharing opportunities.
If the same sequence of action occurs multiple times, the function
will associate a unique identifier to the sequence.
[share actions] returns a pair [(bindings, lookup) : action list array *
(action list -> int option)]

Expand Down
2 changes: 1 addition & 1 deletion src/reason-parser/reason_parser.mly
Original file line number Diff line number Diff line change
Expand Up @@ -3756,7 +3756,7 @@ field_expr:
*
* After `blah`, the parser couldn't tell whether to reduce `label` or
* `val_ident`. So inlining the terminal here to avoid the whole decision.
* Another approach would have been to place the `label` rule at at a precedence
* Another approach would have been to place the `label` rule at a precedence
* of below_COLON or something.
*)
| as_loc(LIDENT) COLON expr
Expand Down
2 changes: 1 addition & 1 deletion test/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,7 @@ Make sure the OCaml's version is between the range of the `ocaml` field in esy.j
1. Install the specific version you want with: `esy add ocaml@{{ ocaml_version }}`
2. Run the test with: `esy dune runtest`

### Setup the local enviroment with opam
### Setup the local environment with opam

The opam workflow is only tested in CI and isn't needed for development, but can become handy since you can install many switches with different versions of OCaml and load them when needed.

Expand Down