jaredly / jaredly/get_in_ppx

Inline records not working with get_in_ppx?

Open
#13 0 comments 2 reactions 0 assignees View on GitHub
Dominant language
Reason
Stars
40
Forks
3
PR merge metrics
No merged PRs in 30d

Description

Starting with a clean project `bsb -init my-new-project -theme basic-reason`, using an inline record seems to work fine:
```
type inlineRecordTest =
| Inline({test: string});

Js.log("Hello, BuckleScript and Reason!");
```

But as soon as I add `get_in_ppx` the compiler fails with:
```
>>>> Start compiling
[1/3] Building src/Demo.reast
FAILED: src/Demo.reast
/pathToProject/node_modules/bs-platform/lib/bsc.exe -w -30-40+6+7+27+32..39+44+45+101 -warn-error +101 -color always -ppx /pathToProject/node_modules/get_in_ppx/ppx -o src/Demo.reast -bs-syntax-only -bs-binary-ast /pathToProject/src/Demo.re
Fatal error: exception File "/pathToProject/src/Demo.re", line 2, characters 4-10: inline records are not supported before OCaml 4.03

We've found a bug for you!
/userPath/src/Demo.re

Error while running external preprocessor
Command line: /pathToProject/node_modules/get_in_ppx/ppx '/var/folders/c4/sn_mx1cx3dld13k_l6v3v4f40000gn/T/camlppx13b40e' '/var/folders/c4/sn_mx1cx3dld13k_l6v3v4f40000gn/T/camlppx3abfc8'
```

Contributor guide

No contributing guide indexed for this repository

Research direction

Reproduce the failure from the clean `bsb -init my-new-project -theme basic-reason` project using the inline record in `src/Demo.re` and `get_in_ppx/ppx`. Start by inspecting the external preprocessor invocation and its handling of inline records. Done means the example compiles successfully with `get_in_ppx` enabled.

Written by the indexing model from the issue text.

Assessment

Tech stack
ocaml
Domain
build-system
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.