DEPRECATED

This package was republished as jfmengels/elm-review-common in order to have a more consistent naming convention for elm-review rule packages.

To migrate, I recommend going to your review configuration and running the following commands:

# NOTE: You'll need to have Node.js installed to be able to use `npx`
npx elm-json uninstall jfmengels/review-common --yes
npx elm-json install jfmengels/elm-review-common --yes

review-common

Provides common linting rules for elm-review.

Provided rules

• NoExposingEverything - Forbids exporting everything from a module.
• NoImportingEverything - Forbids importing everything from a module.
• NoMissingTypeAnnotation - Reports top-level declarations that do not have a type annotation.
• NoMissingTypeAnnotationInLetIn - Reports let in declarations that do not have a type annotation.
• NoMissingTypeExpose - Reports types that should be exposed but are not.

Configuration

import NoExposingEverything
import NoImportingEverything
import NoMissingTypeAnnotation
import NoMissingTypeAnnotationInLetIn
import NoMissingTypeExpose
import Review.Rule exposing (Rule)

config : List Rule
config =
    [ NoExposingEverything.rule
    , NoImportingEverything.rule []
    , NoMissingTypeAnnotation.rule
    , NoMissingTypeAnnotationInLetIn.rule
    , NoMissingTypeExpose.rule
    ]

Thanks

Thanks to @sparksp for writing NoMissingTypeExpose.