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

manovotny / eslint-config-get-off-my-lawn

A highly opinionated, sharable config of ESLint rules to produce beautiful, readable JavaScript.

MIT License

55 stars 8 forks source link

Open bmitchinson opened 5 years ago

bmitchinson commented 5 years ago

Current Open Issue - Here

Wondering if that's enough to remove it from beta of v5 until it's fixed? Currently throwing unnecessary errors for Animated.Text components.

manovotny commented 5 years ago

I am open to suggestions... Should we remove the rule completely or add the suggested work around?

bmitchinson commented 5 years ago

Mostly just wanted to bring its instability into attention, could go either way. 🤷‍♂

Gut feeling would be to disable any rule that's not stable in all contexts.

manovotny commented 5 years ago

I agree with you. It's probably wise to wait for stability.

It's been removed in 5.0.0-beta.6.

manovotny commented 3 years ago

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