eslint: Use "onlyDeclarations": true in rule id-match

And take advantage of this in the one existing spot where this is
applicable.

Author: chrisbobbe

.eslintrc.yaml

@@ -208,8 +208,20 @@ rules:
   # [1]: https://github.com/tc39/proposal-class-fields
   no-underscore-dangle: ["error", { allowAfterThis: true } ]
 
-  # Disallow double underscores and trailing underscores.
-  id-match: ["error", "^_?([a-zA-Z0-9$]+_?)*$", { "properties": true }]
+  # Disallow names with double underscores and trailing underscores.
+  id-match:
+    - error
+    - "^_?([a-zA-Z0-9$]+_?)*$"
+    - # Don't apply to property names; see comment on no-underscore-dangle.
+      properties: true
+
+      # Only apply to declarations, not uses.  That way a local override for
+      # a given name only needs to appear once, not at each use.
+      #
+      # The docs (https://eslint.org/docs/latest/rules/id-match) say this
+      # only applies to `var`, `function`, and `class` declarations, but in
+      # fact it seems to correctly apply to `const` and other declarations.
+      onlyDeclarations: true
 
 
   #

src/styles/index.js

@@ -1,5 +1,5 @@
 /* @flow strict-local */
-/* eslint-disable id-match */
+// eslint-disable-next-line id-match
 import type { ____Styles_Internal } from 'react-native/Libraries/StyleSheet/StyleSheetTypes';
 
 import composeBoxStyles from './composeBoxStyles';