ArkScript-lang / ArkScript-lang/modules

Switch nlohmann/json for rapidjson

Aberta
#59 0 comentários 0 reações 0 responsáveis Ver no GitHub
good first issue help wanted priority/medium
Linguagem predominante
C++
Estrelas
7
Forks
4
Métricas de merge de PRs
Nenhum PR com merge em 30d

Descrição

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.

Guia de contribuição

Nenhum guia de contribuição indexado para este repositório

Direção de pesquisa

Comece localizando o módulo json e seu submódulo nlohmann/json atual; revise como os testes exercitam sua API e como as dependências são instaladas por meio do CPM. Substitua a dependência sem alterar a API do módulo, remova o submódulo e confirme que os testes existentes passam; adicione testes quando necessário.

Escrita pelo modelo de indexação a partir do texto da issue.

Avaliação

Stack de tecnologia
cpp
Domínio
backend
Tipo de issue
Refatoração
Dificuldade
4/5
Tempo estimado
3-5 dias
Status de atividade
Estagnada
Clareza
Razoavelmente clara
Facilidade para iniciantes
25/100

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.