finos/morphir-elm

Remove Field from value expressions in the IR and use FieldFunction with Apply as a replacement

Aberta

#156 aberto em 17 de set. de 2020

Ver no GitHubGuia do colaborador
 (0 comentário) (0 reação) (0 responsável)Elm (69 forks)auto 404
enhancementgood first issueir-simplification

Métricas do repositório

Stars
 (51 estrelas)
Métricas de merge de PR
 (Mesclagem média 3h 4m) (2 fundiu PRs em 30d)

Description

Currently the IR has both Field and FieldFunction as values in the IR. The corresponding example in Elm would be a.foo vs. .foo. The idea is that a.foo can be represented as .foo a (applying the .foo function to a). The latter is more generic and the former is basically just a special combination of FieldFunction and Apply therefore it's accidental complexity which should be removed.

The specific change is to remove Field: https://github.com/finos/morphir-elm/blob/5e18e70bd4631aea1d1b677be04b62de3069a5eb/src/Morphir/IR/Value.elm#L90

And replace throughout the codebase using the following substitution rule:

Field a subjectValue fieldName -> Apply a (FieldFunction a fieldName) subjectValue

Guia do colaborador

Direção de pesquisa
Identifique todas as ocorrências de `Field` no código, entenda a regra de substituição e substitua cada uma por `Apply (FieldFunction a fieldName) subjectValue`. Certifique se de que o IR e todas as transformações relacionadas sejam atualizadas de forma consistente.
Pilha de tecnologia
None
Domain
compilers
Tipo Issue
Refatoração
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.
Recente
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
ElmGit
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
30