Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork2.8k
Open
Labels
Description
Is there an existing issue for this?
- I have searched the existing issues and my issue is unique
- My issue appears in the command-line and not only in the text editor
Description Overview
Tl;dr the lint rule doesn't pick up items that are inserted into an array via push/unshift etc.
Expected Behavior
I've seen this pattern many times and I think we could reasonably assume that when the method name matches an array insertion method we can report an error.
This isn't foolproof without type information but I think this will be valuable.
Example:
constComp=()=>{constitems=[];if(someCondition){// this should cause an error/warningitems.push(<span>test</span>);}if(someOtherCondition){// same items.push(<span>test2</span>);}return<div>{items}</div>;}
eslint-plugin-react version
7.37.5
eslint version
9.30.1
node version
24