bump AST to 4.11 - Githubissues

reasonml / reason

Simple, fast & type safe code that leverages the JavaScript & OCaml ecosystems

http://reasonml.github.io

MIT License

10.08k stars 425 forks source link

bump AST to 4.11 #2657

Closed EduardoRFS closed 2 years ago

EduardoRFS commented 2 years ago

Why

If we plan to support new features our internal AST needs to support it.

Why 4.11?

4.11 is the next biggest update on the AST level, so it's a good target for an incremental change, 4.12 also adds injectivity which would require us to change the parser in more detail.

anmonteiro commented 2 years ago

We've been testing this for a little while and it feels like a safe change to merge.

I'm gonna press "Squash and merge" in case we need to revert it later.