feat: reduce too many exposes module in toplevel#773
Conversation
|
Instead of going to that direction, probably |
|
@davesnx I'm not familiar with term of |
|
With |
We'll need both. This PR alone still exposes all the other modules because there's |
| (preprocess | ||
| (pps melange.ppx reason-react-ppx)) | ||
| (wrapped false) | ||
| (wrapped true) |
There was a problem hiding this comment.
I have been changes to be true @davesnx . I think is incomplete yet? could you give me suggestions for this changes?
There was a problem hiding this comment.
One way to move forward with this would be to fix the build
|
@davesnx do you want to include this for 0.12, or should it be included after that? |
|
Let's do it for 0.13 |
|
This is actually going to be a lot harder than initially thought, since many modules depend on the |
|
solution in #783 |
|
I learn a lot from #783 and I closed this PRs Good job! Thanks @anmonteiro |
Based on this #770