third-party JSON dependency included in header
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 3/5
- Temps estimé
- 1-2 jours
- Accessibilité débutants
- 35/100
- Type d'issue
- Refactorisation
- Clarté
- Plutôt claire
- Activité
- À l'abandon
- Stack technique
- cpp
- Domaine
- build-system
Piste de recherche
Examinez les en-têtes publics et les fichiers d’implémentation du SDK à la recherche de l’inclusion de nlohmann/json et du chemin de dépendance codé en dur. Déplacez l’utilisation de la dépendance dans l’implémentation .cpp afin que les clients ne soient pas obligés d’inclure ou de résoudre nlohmann/json via l’en-tête.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
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
- Langage dominant
- C++
- Étoiles
- 34
- Forks
- 25
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Préparer son environnement
Ce projet ne fournit ni conteneur de développement, ni Dockerfile, ni guide de contribution : l'installation est à votre charge. Commencez par son README, et consultez notre guide de la première contribution pour les étapes générales.
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de Countly/countly-sdk-cpp
-
Difficulté 2/5 1-3 heures Accessibilité débutants 45/100
Countly/countly-sdk-cpp#103 · 3 commentaires ·
-
Difficulté 4/5 3-5 jours Accessibilité débutants 45/100
Countly/countly-sdk-cpp#100 ·
-
Difficulté 3/5 1-2 jours Accessibilité débutants 35/100
Countly/countly-sdk-cpp#36 · 4 commentaires ·
-
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 30/100
Countly/countly-sdk-cpp#35 · 3 commentaires ·
-
No namespaceOuverte
Difficulté 2/5 1-3 heures Accessibilité débutants 35/100
Countly/countly-sdk-cpp#33 · 1 réaction ·
Toutes les issues de Countly/countly-sdk-cpp
Issues similaires
-
Mac mini M6 (16 GB): Q2_0 streaming works - measured 11.7->27.2 tok/s + arena fix for sub-32 GB MacsOuverte
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
Les mainteneurs répondent en général sous 1 jour
-
champion good first issue remake source/qq
Difficulté 2/5 1-3 heures Accessibilité débutants 65/100
metasequoiaime/msime-windows#1025 ·
Les mainteneurs répondent en général sous 1 jour
-
backlog bug
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
EchoTools/nevr-runtime#653 ·
Les mainteneurs répondent en général sous 1 jour
-
`--fail-require-skip` never fires: run_tests.py searches stdout, but the unittest binary writes the skip summary to stderrPeut-être pris Une pull request liée à cette issue est ouverte ou déjà fusionnée. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
InfiniTimeOrg/InfiniTime#2492 ·