Align Microsoft.ReactNative.Cxx includes with core public headers
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 42/100
- Tipo de issue
- Refactorización
- Claridad
- Bastante claro
- Estado de actividad
- Activo
- Stack tecnológico
- cpp, react-native
- Área
- build-system
Línea de trabajo
Read Microsoft.ReactNative.Cxx.vcxitems alongside the vnext/include/React forwarders and the core <React/*.h> umbrellas, using PR #16513 as the reference for the intended public include shape. Inventory the header closure each umbrella needs, then compare it with the Cxx include path and ClInclude entries. Done means ReactCommon is no longer on the public include path, the NuGet ships only that closure, and exclusions such as Dynamic.h (#11644) remain.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Problem Description
Core now treats <React/*.h> as the public C++ entry point and guards direct includes of fine-grained headers. vnext/include/React already forwards those umbrellas, but Microsoft.ReactNative.Cxx.vcxitems still adds all of ReactCommon to the public include path and lists private headers as package includes. Public headers therefore pull in private headers, and the NuGet does not ship the include/React forwarders.
Follow-up from https://github.com/microsoft/react-native-windows/pull/16513. MS.RN.Cxx should match core's public includes.
Scope:
- Inventory the header closure required by each core
<React/*.h>umbrella. - Compare that closure with the Cxx include path and
ClIncludeentries. - Stop exposing
ReactCommonon the public include path. - Ship only that closure in the Microsoft.ReactNative.Cxx NuGet.
- Keep the existing Windows exclusions, including
Dynamic.h(#11644). - Do not move headers out of the npm package until that layout is agreed.
Removing ReactCommon from the include path before the closure is packaged would break in-tree and external native code.
Integration PR: https://github.com/microsoft/react-native-windows/pull/16513
Nightly: 0.89.0-nightly-20261002-cd2de187e
- Lenguaje dominante
- C++
- Estrellas
- 17.4k
- Forks
- 1.2k
- Merge medio
- 1 d 20 h
- PR fusionados (30 d)
- 37
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Tiene una plantilla de pull request
- Leer la guía de contribución
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 microsoft/react-native-windows
-
Needs: Triage :mag:
Dificultad 2/5 1-2 días Aptitud para principiantes 64/100
microsoft/react-native-windows#16527 ·
Los mantenedores suelen responder en 1 día
-
Fabric text is drawn with ClearType onto transparent composition surfaces, fringing thin glyphsAbiertoNeeds: Triage :mag:
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
microsoft/react-native-windows#16340 ·
Los mantenedores suelen responder en 1 día
-
bug Needs: Triage :mag:
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
microsoft/react-native-windows#16321 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
bug Needs: Triage :mag:
Dificultad 3/5 1-2 días Aptitud para principiantes 48/100
microsoft/react-native-windows#16479 ·
Los mantenedores suelen responder en 1 día
-
Needs: Triage :mag:
Dificultad 5/5 Más de una semana Aptitud para principiantes 30/100
microsoft/react-native-windows#16442 · 2 comentarios ·
Los mantenedores suelen responder en 1 día
Todos los issues de microsoft/react-native-windows
Issues similares
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 88/100
ROCm/FastFlowLM#887 ·
Los mantenedores suelen responder en 1 día
-
paraview 6.2.0 uses Abseil 20260526 symbols but does not pin libabseil (undefined symbol with libabseil 20260817)Posiblemente ocupada @jschueller la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 64/100
-
bug medium priority openmp tokenizer wrong result
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 92/100
apache/casbin-cpp#277 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
iamaisim/ProjectAirSim#241 ·
Los mantenedores suelen responder en 1 día