This repository has been archived by the owner on Jan 13, 2023. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 140
Add eslint-plugin-react-hooks #274
Labels
Comments
Can i do this ? |
kamilamin
added a commit
to kamilamin/ignite-bowser
that referenced
this issue
Oct 18, 2019
@kamilamin Sure! |
nonameolsson
added a commit
to nonameolsson/ignite-bowser
that referenced
this issue
Aug 18, 2020
nonameolsson
added a commit
to nonameolsson/ignite-bowser
that referenced
this issue
Aug 18, 2020
infinitered-circleci
pushed a commit
that referenced
this issue
Sep 24, 2020
# [5.4.0](v5.3.0...v5.4.0) (2020-09-24) ### Bug Fixes * **expo:** Update references when using Expo -- fixes [#370](#370) ([#371](#371) by [@nonameolsson](https://github.com/nonameolsson)) ([353daf5](353daf5)) * **hooks:** Fix [#274](#274) - Add eslint-plugin-react-hooks ([#369](#369) by [@nonameolsson](https://github.com/nonameolsson)) ([d08a27b](d08a27b)) * **types:** Add types for i18n ([#365](#365) by [@premithk](https://github.com/premithk)) ([91bfad7](91bfad7)) ### Features * **yellowbox:** Use YellowBox with Expo -- fixes [#367](#367) ([#368](#368) by [@nonameolsson](https://github.com/nonameolsson)) ([fb03e55](fb03e55))
🎉 This issue has been resolved in version 5.4.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Now that we've moved to using React hooks for local state (#239), it would be great to include the ESLint plugin
eslint-plugin-react-hooks
to avoid common hooks pitfalls.https://www.npmjs.com/package/eslint-plugin-react-hooks
For first-time contributors, you'll likely be working in here:
ignite-bowser/boilerplate/package.json.ejs
Line 48 in 726ffb1
And maybe adding more to the ESLint configuration in there.
The text was updated successfully, but these errors were encountered: