SauveJeanLuc / coffee-tasks

Next gen AI Powered Todo-List
https://coffeetasks.netlify.app
MIT License
38 stars 73 forks source link

Add new eslint rule which removes the need to have React in scope #128

Closed gaber85 closed 1 year ago

gaber85 commented 1 year ago

Description

As of React 17, the need to have React in scope is no longer necessary. JSX Transform was introduced and it takes care of injecting react for us. This ESLint rule removes the warning you would normally get when React is not in scope.

I've also changed all the existing files to remove React as well.

For more information on this subject, be sure to check the blog on JSX Transform.

Comment on the PR if you have any questions.

Motivation and Context

This change will bring the repo to use the new tech available!

Checklist:

netlify[bot] commented 1 year ago

Deploy Preview for coffeetasks ready!

Name Link
Latest commit 3a924e9c0e182d147eb66b6ed51cbd01b3aba8f1
Latest deploy log https://app.netlify.com/sites/coffeetasks/deploys/63497866eb7e020007a7aa35
Deploy Preview https://deploy-preview-128--coffeetasks.netlify.app/
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

SauveJeanLuc commented 1 year ago

pull latest changes from master

gaber85 commented 1 year ago

@SauveJeanLuc I've merged master now. All fast forwarded and passing!