third-party JSON dependency included in header
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 35/100
- Tipo de issue
- Refactorización
- Claridad
- Bastante claro
- Estado de actividad
- Estancado
- Stack tecnológico
- cpp
- Área
- build-system
Línea de trabajo
Inspecciona los encabezados públicos y los archivos de implementación del SDK en busca de la inclusión de nlohmann/json y de la ruta de dependencia codificada de forma fija. Traslada el uso de la dependencia a la implementación .cpp para que los clientes no tengan que incluir ni resolver nlohmann/json a través del encabezado.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
The third party dependency nlohmann/json is included in the header, which means clients of this SDK will also be pulling it in. this can conflict with a different but existing version of that library, and also means if a client is using a different JSON library, it now depends on multiple JSON libs. the path specified for that JSON dependency is also hardcoded
it would be great to move this dependency to the .cpp so clients don't need to worry about it
- Lenguaje dominante
- C++
- Estrellas
- 34
- Forks
- 25
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.
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 Countly/countly-sdk-cpp
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 45/100
Countly/countly-sdk-cpp#103 · 3 comentarios ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 45/100
Countly/countly-sdk-cpp#100 ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 35/100
Countly/countly-sdk-cpp#36 · 4 comentarios ·
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 30/100
Countly/countly-sdk-cpp#35 · 3 comentarios ·
-
No namespaceAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 35/100
Countly/countly-sdk-cpp#33 · 1 reacción ·
Todos los issues de Countly/countly-sdk-cpp
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
Los mantenedores suelen responder en 2 días
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
eclipse-score/inc_someip_gateway#356 ·
Los mantenedores suelen responder en 2 días
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
HarperFast/rocksdb-js#945 ·
Los mantenedores suelen responder en 1 día
-
A failed COMMIT leaves the DuckLake transaction registered, leaking the whole DuckDB instanceAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 1 día
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 85/100
Los mantenedores suelen responder en 1 día