ArkScript-lang / ArkScript-lang/modules

Switch nlohmann/json for rapidjson

Ouverte
#59 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub
good first issue help wanted priority/medium
Langage dominant
C++
Étoiles
7
Forks
4
Métriques de merge des PR
Aucune PR mergée en 30 j

Description

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.

Guide de contribution

Aucun guide de contribution indexé pour ce dépôt

Piste de recherche

Commencez par localiser le module json et son sous-module nlohmann/json actuel ; examinez comment les tests utilisent son API et comment les dépendances sont installées via CPM. Remplacez la dépendance sans modifier l’API du module, supprimez le sous-module et vérifiez que les tests existants passent ; ajoutez des tests si nécessaire.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
cpp
Domaine
backend
Type d'issue
Refactorisation
Difficulté
4/5
Temps estimé
3-5 jours
Activité
À l'abandon
Clarté
Plutôt claire
Accessibilité débutants
25/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.