Add OverloadedLists caveat note to "Use :" hint (fixes #1602)

With OverloadedLists enabled, `\x -> [x]` and `(: [])` may have
different types: the former can produce any IsList instance
(Vector, Set, NonEmpty, ...) while the latter specialises to [a].
Applying the suggestion then produces code that no longer type-checks.

Per the maintainer's response on #1602, auto-suppression is not
warranted (OverloadedLists is uncommon), but an explanatory note is.

Author: mgajda

data/hlint.yaml

@@ -211,7 +211,7 @@
     - hint: {lhs: any (const True) x, rhs: not (null x), name: Use null}
     - hint: {lhs: length x /= 0, rhs: not (null x), note: IncreasesLaziness, name: Use null}
     - hint: {lhs: 0 /= length x, rhs: not (null x), note: IncreasesLaziness, name: Use null}
-    - hint: {lhs: "\\x -> [x]", rhs: "(:[])", name: "Use :"}
+    - hint: {lhs: "\\x -> [x]", rhs: "(:[])", name: "Use :", note: "incorrect if OverloadedLists is enabled"}
     - hint: {lhs: map f (zip x y), rhs: zipWith (curry f) x y, side: not (isApp f)}
     - hint: {lhs: "map f (fromMaybe [] x)", rhs: "maybe [] (map f) x"}
     - hint: {lhs: "concatMap f (fromMaybe [] x)", rhs: "maybe [] (concatMap f) x"}