Michael Novotny comments

Results 46 comments of


                                            Michael Novotny

Consider adding `jsx-a11y/control-has-associated-label` when it becomes more stable

Both issues still open. Still holding off on adoption.

Consider adding `jsx-a11y/control-has-associated-label` when it becomes more stable

@zourdycodes can you share your entire configuration for the rule?

Possibly replace `eslint-plugin-security` with `eslint-plugin-security-node`

@gkouziik I'd recommend chiming in on the [latest thread](https://github.com/nodesecurity/eslint-plugin-security/issues/71). There seems to be some traction and I'd love to see all of this come into some long-term ownership.

Consider adding `eslint-plugin-get-off-my-grassql`

Might be worth waiting until #37.

Consider adding `react-native/no-raw-text` when when it becomes more stable

I am open to suggestions... Should we remove the rule completely or [add the suggested work around](https://github.com/Intellicode/eslint-plugin-react-native/issues/225#issuecomment-480249263)?

Consider adding `react-native/no-raw-text` when when it becomes more stable

I agree with you. It's probably wise to wait for stability. It's been removed in `5.0.0-beta.6`.

Consider adding `react-native/no-raw-text` when when it becomes more stable

Upstream issue still exists. Still holding on adoption for now.

Consider adding `require-atomic-updates` when it becomes more stable

@roboli yep, that's exactly why I removed it from v5 after beta feedback. I just released `5.0.0-beta.5`, which no longer contains the rule.

Consider adding `require-atomic-updates` when it becomes more stable

Updated original message. Two of three issues fixed! Progress, but still holding off on adoption for now.

Consider adding `node/no-unsupported-features/es-syntax`

Attempting to add in #133.