third-party JSON dependency included in header
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Anfängerfreundlichkeit
- 35/100
- Issue-Typ
- Refactoring
- Klarheit
- Größtenteils klar
- Aktivitätsstatus
- Veraltet
- Tech-Stack
- cpp
- Bereich
- build-system
Rechercherichtung
Untersuche die öffentlichen Header und Implementierungsdateien des SDK auf das nlohmann/json-Include und den fest codierten Abhängigkeitspfad. Verlagere die Verwendung der Abhängigkeit in die .cpp-Implementierung, damit Clients nicht gezwungen sind, nlohmann/json über den Header einzubinden oder aufzulösen.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
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
- Vorherrschende Sprache
- C++
- Sterne
- 34
- Forks
- 25
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Entwicklungsumgebung
Dieses Projekt bietet weder Dev-Container noch Dockerfile noch Beitragsleitfaden – die Einrichtung liegt bei Ihnen. Beginnen Sie mit der README; die allgemeinen Schritte stehen in unserem Leitfaden für den ersten Beitrag.
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 Countly/countly-sdk-cpp
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 45/100
Countly/countly-sdk-cpp#103 · 3 Kommentare ·
-
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 45/100
Countly/countly-sdk-cpp#100 ·
-
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 35/100
Countly/countly-sdk-cpp#36 · 4 Kommentare ·
-
Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 30/100
Countly/countly-sdk-cpp#35 · 3 Kommentare ·
-
No namespaceOffen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 35/100
Countly/countly-sdk-cpp#33 · 1 Reaktion ·
Alle Issues in Countly/countly-sdk-cpp
Ähnliche Issues
-
bug
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 82/100
Maintainer antworten meist innerhalb von 1 Tag
-
Mac mini M6 (16 GB): Q2_0 streaming works - measured 11.7->27.2 tok/s + arena fix for sub-32 GB MacsOffen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
Maintainer antworten meist innerhalb von 1 Tag
-
champion good first issue remake source/qq
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 65/100
metasequoiaime/msime-windows#1025 ·
Maintainer antworten meist innerhalb von 1 Tag
-
backlog bug
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 75/100
EchoTools/nevr-runtime#653 ·
Maintainer antworten meist innerhalb von 1 Tag
-
`--fail-require-skip` never fires: run_tests.py searches stdout, but the unittest binary writes the skip summary to stderrEvtl. vergeben Ein verknüpfter Pull Request ist offen oder bereits gemergt. Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 72/100
Maintainer antworten meist innerhalb von 1 Tag