Skip to content

Add elm-review rule NoUnused.Dependencies

Frank Tackitt requested to merge kageurufu/exosphere:elm-review/NoUnused.Dependencies into master

Overview

Add elm-review rule NoUnused.Dependencies

We don't have any unused dependencies, but it would look like

-- ELM-REVIEW ERROR --------------------------------------------- elm.json:30:14

(fix) NoUnused.Dependencies: Unused dependency `jfmengels/elm-review-unused`

29|             "hermanverschooten/ip": "1.0.1",
30|             "jfmengels/elm-review-unused": "1.2.0",
                    ^^^^^^^^^^^^^^^^^^^^^^^^^^^
31|             "justinmimbs/time-extra": "1.1.1",

To remove it, I recommend running the following command:

    elm-json uninstall jfmengels/elm-review-unused

Discussion

I like this. It just prevents future code smell.

Merge request reports