jfmengels/elm-review-simplify

Merging of let..in can lead to naming collisions

Open

#313 ouverte le 26 juin 2024

Voir sur GitHub
 (0 commentaires) (1 réaction) (0 assignés)Elm (10 forks)github user discovery
bughelp wanted

Métriques du dépôt

Stars
 (22 stars)
Métriques de merge PR
 (Métriques PR en attente)

Description

SSCCE

value =
  let
    a name = name
  in
  let
    name = "str"
  in
  a name

Currently, this gets simplified to the following, which introduces a naming collision error from the compiler.

value =
  let
    a name = name

    name = "str"
  in
  a name

I think the solution would be to check that there will be no collisions, but this is quite a bit of analysis compared to what is done today. I wonder if there's a smart way of doing it :thinking:

Guide contributeur

Direction de recherche
Comprenez comment les expressions let..in sont simplifiées et implémentez la détection de collisions avant de les fusionner.
Stack technique
Aucun
Domaine
tooling
Type d'issue
Bug
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.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
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.
65