finos/morphir-elm

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

Ouverte

#156 ouverte le 17 sept. 2020

Voir sur GitHubGuide contributeur
 (0 commentaire) (0 réaction) (0 personne assignée)Elm (69 forks)auto 404
enhancementgood first issueir-simplification

Métriques du dépôt

Stars
 (51 étoiles)
Métriques de merge PR
 (Merge moyen 3h 4m) (2 PRs mergées en 30 j)

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

Guide contributeur

Direction de recherche
Identifiez toutes les occurrences de `Field` dans le code, comprenez la règle de substitution et remplacez chacune par `Apply (FieldFunction a fieldName) subjectValue`. Assurez vous que l'IR et toutes les transformations associées sont mises à jour de manière cohérente.
Stack technique
Aucun
Domaine
compilers
Type d'issue
Refactorisation
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-2 jours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Récente
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
ElmGit
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
30