LinusU / LinusU/react-native-get-random-values
Add support for Windows
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- JavaScript
- Estrellas
- 415
- Forks
- 63
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Descripción
With React Native for Windows becoming a viable target, enabling this package for Windows would be a great benefit to the community! (Especially if this package backs uuid, since a library I plan on using in RN uses it.)
I'm willing to take a stab at an initial implementation, and wanted to post this issue as a place to track progress.
Todo:
-
Ensure RN for Windows supports synchronous methodsIt does! -
Decide if C++ or C# would be the most appropriateC# would be the most approachable, and would be just as performant as the C++ version since they're both just calling UWP APIs -
Determine if RN prefers Win32, UWP, or both for third party modulesRN for Windows only works on Windows 10, so there's no sense in using the Win32 APIs. - Setup project (blocked by https://github.com/microsoft/react-native-windows/issues/4211)
- Rewrite module in C++ (blocked by https://github.com/microsoft/react-native-windows/issues/4215)
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.
Línea de trabajo
No se nombran archivos, pruebas ni puntos de entrada. Revisa primero las decisiones de configuración completadas y los bloqueadores vinculados de react-native-windows; el issue está terminado cuando el módulo tenga una implementación de C++ funcional compatible con Windows.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- cpp, csharp, react-native
- Área
- desktop-dev, mobile-dev
- Tipo de issue
- Nueva funcionalidad
- Dificultad
- 5/5
- Tiempo estimado
- Más de una semana
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 25/100