bluesky-social / social-app

The Bluesky Social application for Web, iOS, and Android
https://bsky.app
MIT License
12.65k stars 1.61k forks source link

Leverage jsx transform to remove unnecessary react imports and update eslint config #6516

Closed PaulACoroneos closed 6 days ago

PaulACoroneos commented 6 days ago

Per feedback in #6494 we can break up migrating to ESLint v9 into smaller pieces. One thing I noticed is that bsky.social has not yet implemented the JSX transform introduced by the React team.. It is no longer necessary to import React in JSX components if React itself is not being referenced.

The babel transform is built and in default as part of babel-preset-expo and is turned to automatic by default.

In addition per react.dev blog, the JSX transform is mandatory to move to React 19.

What I did was go ahead and extend the eslint config to include the jsx-runtime configuration and I have also disabled prop-types since we use Typescript. I ran yarn web to validate the application runs after this change. Please let me know if any further details are required.