Idan Entin
Idan Entin
Tried a PR adding a test like: ``` test('reasonable performance', () => { const {getByRole, getByLabelText} = render( ` check`, ) let start = Date.now() getByLabelText(/check/i) let end = Date.now()...
I totally agree with @eps1lon , but if there is something to do to improve performance we should give it a try, shouldn't we? The difference between `getByRole` and `getByLabelText`...
Maybe just add type definitions? Personally I believe TS can be a blocker for new devs, but there are ways to achieve type safety without changing all the way, right?
I feel like the "other side" wasn't heard enough. Most of the pro TS here just dismiss the fact that TS is a hurdle that will push non-TS devs away....
@tmpvar @jsakas , any chance someone can look into this anytime soon?
@domenic maybe? We really need this PR
@tmpvar @jsakas friendly reminder?
I used [`react-live`](https://www.npmjs.com/package/react-live) in a react specimen. Worked pretty nice. I wanted to submit a PR, but I had some issues building catalog locally.
Sure. Do you have some in mind?
That's much more difficult... Do you think we should use a different "entry" than what is currently used for this rule? I don't really have any experience or knowledge in...