NoUnused.Variables: False negative for imports shadowed by another import

jfmengels / elm-review-unused

Provides elm-review rules to detect unused elements in your Elm project

BSD 3-Clause "New" or "Revised" License

23 stars 12 forks source link

Closed jfmengels closed 2 years ago

jfmengels commented 2 years ago

Describe the bug Imports that are shadowed by another import are not being reported

SSCCE (Short Self-Contained Correct Example)

module A exposing (..)

import Accessibility exposing (Html)
import Html exposing (Html)

a : Html msg
a =
    Html.text ""

Steps to reproduce the behavior: Create a project with both

Expected behavior

The import of Html from Accessibility should be marked as unused.