ArkScript-lang / ArkScript-lang/modules

Switch nlohmann/json for rapidjson

Open
#59 0 comments 0 reactions 0 assignees View on GitHub
good first issue help wanted priority/medium
Dominant language
C++
Stars
7
Forks
4
PR merge metrics
No merged PRs in 30d

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.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by locating the json module and its current nlohmann/json submodule; review how tests exercise its API and how dependencies are installed through CPM. Replace the dependency without changing the module API, remove the submodule, and confirm existing tests pass; add tests where needed.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp
Domain
backend
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.