uber/piranha

PiranhaObjC: Use properties.json to specify configuration properties

Aberta

#51 aberto em 15 de jun. de 2020

Ver no GitHubGuia do colaborador
 (1 comentário) (0 reação) (0 responsável)Java (178 forks)batch import
enhancementgood first issuehelp wantedlegacy-objc

Métricas do repositório

Stars
 (2.143 estrelas)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

Description

Currently PiranhaObjC APIs are hardcoded. See here. Introduce a configuration json file that contains an array of methodProperties with methodName, flagType, returnType and argumentIndex. See below.

{ "methodProperties": [ { "methodName": "isToggleEnabled", "flagType": "treated", "returnType": "boolean", "argumentIndex": 0 }, ... ], }

This properties file will be parsed and the listed APIs will be used for refactoring.

Guia do colaborador

Direção de pesquisa
Revisar as definições de API atualmente codificadas em https://github.com/uber/piranha/blob/master/objc/src/XPFlagRefactoring/XPFlagRefactoring.cpp#L40. Investigar como analisar um arquivo JSON (properties.json) contendo um array de objetos methodProperties com os campos methodName, flagType, returnType e argumentIndex. Examinar as bibliotecas de análise JSON existentes no projeto ou considerar o uso de um analisador JSON C++ leve. Após a análise, substituir as chamadas codificadas pelas propriedades carregadas dinamicamente. A issue tem um comentário, mas nenhuma atividade recente do mantenedor; pode ser útil esclarecer a estrutura JSON esperada e os pontos de integração.
Pilha de tecnologia
objective ccpp
Domain
toolingapi
Tipo Issue
Funcionalidade
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-2 dias
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
JSONC++Objective C
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
50