Remove the requirement for a `react-native-node-api-module` file in prebuilds
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Anfängerfreundlichkeit
- 45/100
- Issue-Typ
- Feature
- Klarheit
- Größtenteils klar
- Aktivitätsstatus
- Veraltet
- Tech-Stack
- react-native, typescript
- Bereich
- build-system, mobile
Rechercherichtung
Lies packages/host/src/node/prebuilds/apple.ts und packages/host/src/node/prebuilds/android.ts sowie das verknüpfte TODO und Issue #42. Lege fest, welches Signal zur automatischen Verknüpfung auf beiden Plattformen verwendet werden soll, aktualisiere die beiden Pfade zum Schreiben der Marker konsistent und überprüfe, dass prebuilds nicht mehr von der entfernten Anforderung abhängen.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
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
- Vorherrschende Sprache
- TypeScript
- Sterne
- 188
- Forks
- 10
- Ø Merge
- 2 T. 17 Std.
- Gemergte PRs (30 T.)
- 3
Beitragsleitfaden
Für dieses Repository ist kein Beitragsleitfaden indexiert
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus callstackincubator/react-native-node-api
-
Android 🤖
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 66/100
-
Sanity check of Hermes source directory via environment variable, when building Android projects OffenAndroid 🤖 good first issue
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
callstackincubator/react-native-node-api#163 · 1 Kommentar ·
-
enhancement good first issue gyp-to-cmake
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 72/100
callstackincubator/react-native-node-api#161 · 3 Kommentare · 1 Reaktion ·
-
enhancement Host 🏡
Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 25/100
-
Automatable good first issue
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 15/100
callstackincubator/react-native-node-api#424 · 1 Kommentar ·
Alle Issues in callstackincubator/react-native-node-api
Ähnliche Issues
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 75/100
vercel-labs/just-bash#464 ·
-
looksLikeSlug() is ASCII-only, so non-Latin entity slugs (e.g. Korean) skip exact match and collapse Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 75/100
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 65/100
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 65/100
-
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 90/100
TanStack/tanstack.com#1293 ·