React JSX PPX umbrella issue

[anmonteiro]

Extracted from this comment

We should make a note to remove the -bs-jsx-3 stuff eventually. Here's my rough plan:

• move the React JSX PPX out of the compiler (Add react-jsx-ppx as a separate package #396)
• distribute this new package with melange
• nudge users to change to (preprocess (pps reactjs_jsx_ppx)) rather than (flags :standard -bs-jsx 3)
• Delete this line in the merlin generated config

[jchavarri]

nudge users to change to (preprocess (pps reactjs_jsx_ppx)) rather than (flags :standard -bs-jsx 3)

as @rgrinberg pointed out, this won't be possible until everything in the ppx (fast pipe, ## operator, etc) are migrated to another melange-ppx, so there is one missing step in the list.

[anmonteiro]

What's the part that's not possible? can't we use the melange ppx without -bs-jsx 3?

[jchavarri]

Sorry, I should have pointed to this item instead:

Delete this line in the merlin generated config

We can't remove the hack in merlin generated config until melange ppx is available independently.

[anmonteiro]

Oh I meant delete the bs-jsx arg from the Merlin config, not the whole line

[jchavarri]

I created #463 to track it.

[davesnx]

Updated #396 without being a breakage for users.

[jchavarri]

I think this issue is now done:

• move the React JSX PPX out of the compiler was done in refactor(reactjs_jsx_ppx): port reactjs ppx to ppxlib / separate package #517
• delete the line in the merlin generated config was done in melange: don't pass -bs-jsx flag to merlin ocaml/dune#7354
• bs-jsx flag was removed in refactor: remove --bs-jsx <version> flag #525