finos/morphir-elm

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

Aperta

#156 aperta il 17 set 2020

Vedi su GitHubGuida contributor
 (0 commenti) (0 reazioni) (0 assegnatari)Elm (69 fork)auto 404
enhancementgood first issueir-simplification

Metriche repository

Star
 (51 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

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

Guida contributor

Direzione di ricerca
Identifica tutte le occorrenze di `Field` nel codice, comprendi la regola di sostituzione e sostituisci ciascuna con `Apply (FieldFunction a fieldName) subjectValue`. Assicurati che l'IR e tutte le trasformazioni correlate siano aggiornate coerentemente.
Tech stack
Nessuno
Dominio
compilers
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-2 giorni
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
ElmGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
30