ArkScript-lang / ArkScript-lang/modules

Switch nlohmann/json for rapidjson

Offen
#59 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
good first issue help wanted priority/medium
Vorherrschende Sprache
C++
Sterne
7
Forks
4
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

We would like to switch to rapidjson (MIT) for the json module, instead of nlohmann/json.

Boost would be faster, but an MIT license is better (benchmarks: https://230.jsondocs.prtest.cppalliance.org/libs/json/doc/html/json/benchmarks.html).

The module API shouldn't change, however more functions could be added, depending on the needs. Tests should pass, more can be added (because we never have enough tests).

The json submodule will have to be removed, rapidjson could be installed through CPM.

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Rechercherichtung

Beginne damit, das json-Modul und sein aktuelles nlohmann/json-Submodul zu lokalisieren; prüfe, wie Tests seine API verwenden und wie Abhängigkeiten über CPM installiert werden. Ersetze die Abhängigkeit, ohne die Modul-API zu ändern, entferne das Submodul und bestätige, dass die vorhandenen Tests erfolgreich sind; füge bei Bedarf Tests hinzu.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
cpp
Bereich
backend
Issue-Typ
Refactoring
Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
25/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.