Skip to content

Commit

Permalink
flowconfig [nfc]: Follow RN in removing deprecated-utility lint.
Browse files Browse the repository at this point in the history
Done now, to follow the template-app change in
facebook/react-native@fcffb961f, on the path to the RN v0.64
upgrade.

That commit reports that this lint rule is no longer needed because
it's turned on by default. This is corroborated by the Flow
changelog for v0.98.0 [1], and I verified it experimentally -- using
`$Supertype` (one of the deprecated utility types this rule is meant
to catch [2]) is flagged as an error with or without this line in
our config.

[1] https://github.com/facebook/flow/blob/master/Changelog.md#0980
[2] https://flow.org/en/docs/linting/rule-reference/#toc-deprecated-utility
  • Loading branch information
chrisbobbe committed Aug 4, 2021
1 parent fc17b21 commit ec10d12
Showing 1 changed file with 0 additions and 1 deletion.
1 change: 0 additions & 1 deletion .flowconfig
Original file line number Diff line number Diff line change
Expand Up @@ -85,7 +85,6 @@ deprecated-type=warn
unsafe-getters-setters=warn
unnecessary-invariant=warn
signature-verification-failure=warn
deprecated-utility=error

; ours
sketchy-number=error
Expand Down

0 comments on commit ec10d12

Please sign in to comment.