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
Parse and print Ppat_open
#2314
Conversation
417099f
to
06d4fdc
Compare
type t = {name: string}; | ||
}; | ||
|
||
let foo = (Foo.{name}) => (); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Want to add some more tests covering the various places patterns can occur? Destructuring tuples, switch statements etc.
30c8e38
to
3dd8e54
Compare
|
||
if [ "$MIN_VERSION" != "$BASE_NAME" ] && [ "$(version "$OCAML_VERSION")" -lt "$(version "$MIN_VERSION")" ] | ||
then | ||
notice " ☒ IGNORED REFMT STEP: Requires OCaml >= $MIN_VERSION" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice touch.
@jordwalke I added some more parsing rules to support parsing e.g. There's only one caveat with the current state of this PR: it parses |
Cool, those seem like nice additions too. Is this ready to merge then? |
@kpsuperplane: The Azure builds are getting an error very similar to one you were getting on reason-native:
Do you recall what the culprit is? How do I fix it? |
@jordwalke this should be ready to merge, @anmonteiro and I looked at it together yesterday. The azure failure is weird, I'm seeing the same problem at #2319 (PR is ready), all tests seems to pass on the other ci's |
@jordwalke I solved it last time by renaming the project to something else (e.g. "facebook-reason"), running a successful build, and renaming it back. Never really figured out the root cause though |
fixes #2301