-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(eslint-plugin): add rules to avoid low performance #118
base: main
Are you sure you want to change the base?
Conversation
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.
Cool sinon !
Je vais me charger de rajouter des explications plus précises
956111b
to
fcfc066
Compare
8147b29
to
92d7a57
Compare
…react-navigation/stack in favor of @react-navigation/native-stack
5d6b985
to
dc465fd
Compare
// This should trigger one error breaking custom FlatList import rule: | ||
// @bam.tech/no-flatlist | ||
|
||
import { FlatList } from "react-native"; |
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.
since we already got tests for all those rules, I'd vote for removing those example files, I think they add noise
What do you think @cyrilbo @matthieugicquel ?
packages/eslint-plugin/lib/rules/no-animated-without-native-driver.ts
Outdated
Show resolved
Hide resolved
f2bd860
to
6c936ae
Compare
…to performance rules
…stom rule for rn svg to show a warning
…o prevent from overriding with the existing import rule
6c936ae
to
71645af
Compare
…k to prevent override with eslint import rule
…cused to prevent from override by import eslint rule
71645af
to
8368a20
Compare
No description provided.