-
Notifications
You must be signed in to change notification settings - Fork 0
Conversation
…nent, update cypress config
…ure/setup-project
✅ Deploy Preview for poc-navmap ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
.eslintrc.js
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
prop-types są domyślnie wymagane przez linta, jak nie to 'react/prop-types'?
dodaj proszę z tego poradnika: https://blog.logrocket.com/12-essential-eslint-rules-react/
- 'plugin:react-hooks/recommended' (info: https://legacy.reactjs.org/docs/hooks-rules.html)
- 'react/no-array-index-key'
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Są wymagane domyślnie
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Co z tymi dwoma powyższymi ustawieniami?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A widzisz nie doczytałem, ten plugin warto dodać, bo chyba go nie ma, a to drugie jest
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Dodałem!
No description provided.