Upgrade vendored Flow parser to 0.320.0 - #8588
Conversation
Signed-off-by: Christoph Knittel <ck@cca.io>
Signed-off-by: Christoph Knittel <ck@cca.io>
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 7091dc2e6b
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
Signed-off-by: Christoph Knittel <ck@cca.io>
Signed-off-by: Christoph Knittel <ck@cca.io>
Signed-off-by: Christoph Knittel <ck@cca.io>
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #8588 +/- ##
=======================================
Coverage 75.91% 75.92%
=======================================
Files 475 475
Lines 63033 63033
=======================================
+ Hits 47853 47859 +6
+ Misses 15180 15174 -6 🚀 New features to boost your workflow:
|
Signed-off-by: Christoph Knittel <ck@cca.io>
rescript
@rescript/belt
@rescript/darwin-arm64
@rescript/darwin-x64
@rescript/linux-arm64
@rescript/linux-x64
@rescript/runtime
@rescript/win32-x64
commit: |
|
Developer playground preview: https://rescript-lang.github.io/rescript/dev-playground/?version=pr-8588 |
|
I guess the flow parser will be aggressively cut to only what's needed by the compiler after this. And that most of it will just go, leaving the rest easy to maintain via agents in future if required. |
Or possibly moved to JSOO's JS parser if feasible. See #8567. |
Summary
This is the second PR in the "Flow Forward" series. Updating the parser separately gives us a stable base before changing the OCaml toolchain.
Related to #8567.