rescript-lang / rescript-lang/rescript-react
Split into multiple packages?
Personne n'a encore pris cette issue.
- Langage dominant
- ReScript
- Étoiles
- 517
- Forks
- 45
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Description
- 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?
Guide de contribution
Aucun guide de contribution indexé pour ce dépôt
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Piste de recherche
Commencez par examiner le contenu actuel du package @rescript/react, ses bindings et dépendances React et react-dom, ainsi que les références à PPX mentionnées dans l’issue. La tâche serait terminée lorsqu’une séparation des packages serait convenue, couvrant la compatibilité avec ReactV3, react-dom et ReScriptReactRouter, y compris la manière dont PPX est affecté.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- react
- Domaine
- frontend
- Type d'issue
- Refactorisation
- Difficulté
- 5/5
- Temps estimé
- Plus d'une semaine
- Activité
- À l'abandon
- Clarté
- À clarifier
- Accessibilité débutants
- 25/100