Remove the requirement for a `react-native-node-api-module` file in prebuilds
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 45/100
- Tipo de issue
- Nueva funcionalidad
- Claridad
- Bastante claro
- Estado de actividad
- Estancado
- Stack tecnológico
- react-native, typescript
- Área
- build-system, mobile
Línea de trabajo
Lee packages/host/src/node/prebuilds/apple.ts y packages/host/src/node/prebuilds/android.ts, junto con el TODO vinculado y el issue #42. Determina qué señal de autoenlazado debe usarse en ambas plataformas, actualiza de forma coherente las dos rutas que escriben los marcadores y verifica que prebuilds ya no dependan del requisito eliminado.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
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
- Lenguaje dominante
- TypeScript
- Estrellas
- 188
- Forks
- 10
- Merge medio
- 2 d 17 h
- PR fusionados (30 d)
- 3
Guía de contribución
No hay ninguna guía de contribución indexada para este repositorio
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de callstackincubator/react-native-node-api
-
Android 🤖
Dificultad 2/5 1-3 horas Aptitud para principiantes 66/100
-
Sanity check of Hermes source directory via environment variable, when building Android projects AbiertoAndroid 🤖 good first issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
callstackincubator/react-native-node-api#163 · 1 comentario ·
-
enhancement good first issue gyp-to-cmake
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
callstackincubator/react-native-node-api#161 · 3 comentarios · 1 reacción ·
-
enhancement Host 🏡
Dificultad 5/5 Más de una semana Aptitud para principiantes 25/100
-
Automatable good first issue
Dificultad 3/5 1-2 días Aptitud para principiantes 15/100
callstackincubator/react-native-node-api#424 · 1 comentario ·
Todos los issues de callstackincubator/react-native-node-api
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
vercel-labs/just-bash#464 ·
-
looksLikeSlug() is ASCII-only, so non-Latin entity slugs (e.g. Korean) skip exact match and collapse Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 90/100
TanStack/tanstack.com#1293 ·