rescript-lang / rescript-lang/rescript-react
Split into multiple packages?
Nessuno ha ancora preso questa issue.
- Lingua principale
- ReScript
- Stelle
- 517
- Fork
- 45
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Descrizione
- With JSX4 bringing breaking changes, the JSX3 bindings have been moved into a separate namespace ReactV3. I was wondering if this ReactV3 namespace shouldn't better be provided by some compatibility package.
@rescript/reactcurrently includes bindings for bothreactandreact-domand also has both as peer and dev dependencies. However, not every React project is also using React DOM. E.g. React Native projects aren't. So IMHOreact-dombindings should go into their own package.- #33 mentions the idea to split out ReScriptReactRouter into a separate package.
If we did all that, we'd end up with something like
@rescript/react@rescript/react-dom@rescript/react-jsx3-compat@rescript/rescript-react-router(or justrescript-react-router?)
@mattdamon108 @cristianoc What do you think? How would such a split affect the PPX?
Guida per i contributori
Nessuna guida per i contributori indicizzata per questo repository
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Direzione di ricerca
Inizia esaminando i contenuti attuali del package @rescript/react, i suoi binding e le sue dipendenze di React e react-dom, nonché i riferimenti a PPX menzionati nell’issue. Il lavoro sarebbe completato quando fosse concordata una suddivisione dei package che copra la compatibilità con ReactV3, react-dom e ReScriptReactRouter, incluso il modo in cui PPX ne è influenzato.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- react
- Ambito
- frontend
- Tipo di issue
- Refactoring
- Difficoltà
- 5/5
- Tempo stimato
- Più di una settimana
- Stato di attività
- Ferma
- Chiarezza
- Da chiarire
- Idoneità per principianti
- 25/100