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.

Sign up for GitHub

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

Jump to bottom

namespaced record field names aren't displayed in formatted output. #201

Open
jordwalke opened this issue Sep 22, 2019 · 1 comment · May be fixed by #259
Open

namespaced record field names aren't displayed in formatted output. #201

jordwalke opened this issue Sep 22, 2019 · 1 comment · May be fixed by #259
Labels
good first issue Good for newcomers

Comments

@jordwalke
Copy link
Member

If there's an unknown record field X.y, the error message says that record field X cannot be found. This should be an easy fix in the parsing/regex.

# Unformatted Error Output:
# File "compiler/js_of_ocaml.ml", line 67, characters 6-19:
# Error: Unbound record field CompileArg.wrap_with_fun


 ERROR  compiler/js_of_ocaml.ml:67 6-19

 64 ┆ ; output_file
 65 ┆ ; params
 66 ┆ ; static_env
 67 ┆ ; wrap_with_fun
 68 ┆ ; dynlink
 69 ┆ ; linkall
 70 ┆ ; toplevel

Record field CompileArg; can't be found in any record type.
@jordwalke jordwalke added the good first issue Good for newcomers label Sep 22, 2019
@ayshiff
Copy link

ayshiff commented Sep 29, 2020 •
edited
Loading

@jordwalke Can I work on this ?

@ayshiff ayshiff linked a pull request Sep 29, 2020 that will close this issue
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
good first issue Good for newcomers
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix: Unbound namespaced record field error ayshiff/reason-native
2 participants
@jordwalke @ayshiff