Remove the requirement for a `react-native-node-api-module` file in prebuilds
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Idoneità per principianti
- 45/100
- Tipo di issue
- Funzionalità
- Chiarezza
- Abbastanza chiara
- Stato di attività
- Ferma
- Stack tecnologico
- react-native, typescript
- Ambito
- build-system, mobile
Direzione di ricerca
Leggi packages/host/src/node/prebuilds/apple.ts e packages/host/src/node/prebuilds/android.ts, insieme al TODO collegato e all’issue #42. Stabilisci quale segnale di auto-linking debba essere usato su entrambe le piattaforme, aggiorna in modo coerente i due percorsi di scrittura dei marker e verifica che prebuilds non dipendano più dal requisito rimosso.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
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
- Lingua principale
- TypeScript
- Stelle
- 188
- Fork
- 10
- Merge medio
- 2g 17h
- PR unite (30g)
- 3
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.
Altre issue di callstackincubator/react-native-node-api
-
Android 🤖
Difficoltà 2/5 1-3 ore Idoneità per principianti 66/100
-
Sanity check of Hermes source directory via environment variable, when building Android projects ApertaAndroid 🤖 good first issue
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
callstackincubator/react-native-node-api#163 · 1 commento ·
-
enhancement good first issue gyp-to-cmake
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
callstackincubator/react-native-node-api#161 · 3 commenti · 1 reazione ·
-
enhancement Host 🏡
Difficoltà 5/5 Più di una settimana Idoneità per principianti 25/100
-
Automatable good first issue
Difficoltà 3/5 1-2 giorni Idoneità per principianti 15/100
callstackincubator/react-native-node-api#424 · 1 commento ·
Tutte le issue di callstackincubator/react-native-node-api
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
vercel-labs/just-bash#464 ·
-
looksLikeSlug() is ASCII-only, so non-Latin entity slugs (e.g. Korean) skip exact match and collapse Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 90/100
TanStack/tanstack.com#1293 ·