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

Enable Uncurried Mode by Default for Parser Development CLI #6765

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

IwanKaramazow
Copy link

@IwanKaramazow IwanKaramazow commented May 23, 2024

When debugging the AST through the parser's development CLI, there is a discrepancy between calling dune exec bsc and dune exec res_parser. The former defaults to uncurried mode, while the latter does not.

This commit enables uncurried mode by default for the parser development CLI. If needed, this can be optionally disabled by using the -legacy-curried flag.

@IwanKaramazow
Copy link
Author

This also influences the parser/printer tests. I'll change this PR to "enable uncurried by default for the parser/printer tests"

Maxim added 2 commits May 24, 2024 15:20
When debugging the AST through the parser's development CLI, there is a discrepancy between calling dune exec bsc and dune exec res_parser. The former defaults to uncurried mode, while the latter does not.

This commit enables uncurried mode by default for the parser development CLI. If needed, this can be optionally disabled by using the -legacy-curried flag.
@IwanKaramazow IwanKaramazow force-pushed the iwan/enable-uncurried-mode-parser-cli-helper branch from 602192e to caf9965 Compare May 24, 2024 13:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant