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

ReasonML support seems to be broken... #404

Closed
Lupus opened this issue Aug 12, 2022 · 1 comment
Closed

ReasonML support seems to be broken... #404

Lupus opened this issue Aug 12, 2022 · 1 comment
Milestone
2.8.3

Comments

@Lupus
Copy link

Lupus commented Aug 12, 2022

For .re sources the report is using .re.ml file, which is a binary AST...
image

bisect_ppx 2.8.1
dune 3.4.1
@aantron aantron modified the milestones: 2.8.2, 2.8.3 Apr 8, 2023
@aantron
Copy link
Owner

aantron commented Jun 25, 2023

Thank you! This either broke at some point due to how locations are set by the Reason setup in Dune, or maybe we never even tested it on Reason+Dune, but only Reason+bsb.

@aantron aantron mentioned this issue Jul 17, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
2.8.3
Development

No branches or pull requests
2 participants
@Lupus @aantron