Change NoReturnArrayVariableListRule to ignore array spreads

[carlos-granados]

Many times we return an array which contains a number of array spreads in order to merge all these arrays and return a single one:

return [...$firstSetOfResults, ...$secondSetOfResults];

The NoReturnArrayVariableListRule sees that we are returning an array composed of two expressions and will return an error asking us to use a value object instead. This is clearly not appropriate for this case

This PR fixes this problem by ignoring array spreads when counting the number of expressions in the returned array

[TomasVotruba]

Looks good 👍 Thanks

[carlos-granados]

Thanks! Do you expect to create a release in this repo in the near future? I am trying to decide if we should use this fork or if we should just ignore the error for now and wait for the release

[TomasVotruba]

Yep, I think next 1-2 weeks tags should land. I'm waiting to collect more fixes/improvement, not to release constantly new versions.

If not, ping me here 👍