Remove the requirement for a `react-native-node-api-module` file in prebuilds
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Accessibilité débutants
- 45/100
- Type d'issue
- Fonctionnalité
- Clarté
- Plutôt claire
- Activité
- À l'abandon
- Stack technique
- react-native, typescript
- Domaine
- build-system, mobile
Piste de recherche
Lisez packages/host/src/node/prebuilds/apple.ts et packages/host/src/node/prebuilds/android.ts, ainsi que le TODO associé et l’issue #42. Déterminez quel signal d’auto-linking doit être utilisé sur les deux plateformes, mettez à jour de manière cohérente les deux chemins d’écriture des marqueurs et vérifiez que prebuilds ne dépendent plus de la prérequis supprimée.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
We currently require that authors of Node-API modules place a "magic" react-native-node-api-module file to signal the prebuild is intended for auto-linking.
When #42 merge we could probably lift this requirement and consider any .apple.node or .android.node directory enabled for auto-linking.
A third option marked in the source
There is a TODO in the Apple prebuild path proposing a different resolution to the same question — moving the marker into the framework's Info.plist rather than dropping it:
Worth settling here rather than in two places, since the options are mutually exclusive. Note the Apple and Android sides differ in what is available: an XCFramework has an Info.plist to carry a custom key, while the Android output is a plain jniLibs directory with nowhere equivalent to put one — so "move it into Info.plist" would leave the two platforms signalling auto-linking in different ways, whereas removing the requirement entirely (this issue as filed) keeps them symmetric.
The marker file is written in two places, both of which would change either way:
packages/host/src/node/prebuilds/apple.ts(linked above)- https://github.com/callstackincubator/react-native-node-api/blob/29a527d87fa8cb965b0048b1b4c1cff2b0eab281/packages/host/src/node/prebuilds/android.ts#L67-L72
- Langage dominant
- TypeScript
- Étoiles
- 188
- Forks
- 10
- Merge moyen
- 2 j 17 h
- PR mergées (30 j)
- 3
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.
Autres issues de callstackincubator/react-native-node-api
-
Android 🤖
Difficulté 2/5 1-3 heures Accessibilité débutants 66/100
-
Sanity check of Hermes source directory via environment variable, when building Android projects OuverteAndroid 🤖 good first issue
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
callstackincubator/react-native-node-api#163 · 1 commentaire ·
-
enhancement good first issue gyp-to-cmake
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
callstackincubator/react-native-node-api#161 · 3 commentaires · 1 réaction ·
-
enhancement Host 🏡
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 25/100
-
Automatable good first issue
Difficulté 3/5 1-2 jours Accessibilité débutants 15/100
callstackincubator/react-native-node-api#424 · 1 commentaire ·
Toutes les issues de callstackincubator/react-native-node-api
Issues similaires
-
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
vercel-labs/just-bash#464 ·
-
looksLikeSlug() is ASCII-only, so non-Latin entity slugs (e.g. Korean) skip exact match and collapse Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
-
Difficulté 2/5 1-3 heures Accessibilité débutants 65/100
-
Difficulté 2/5 1-3 heures Accessibilité débutants 65/100
-
Difficulté 1/5 Moins d'une heure Accessibilité débutants 90/100
TanStack/tanstack.com#1293 ·