array-callback-return should check forEach

What rule do you want to change? array-callback-return

Does this change cause the rule to produce more or fewer warnings? more

How will the change be implemented? (New option, new default behavior, etc.)? new default behavior

Please provide some example code that this change will affect:

[
    "abc",
    "def"
].forEach((word) => {

    "use strict";

    return word;

});

What does the rule currently do for this code? Pass

What will the rule do after it’s changed? Warn that the forEach callback potentially returns a value

Are you willing to submit a pull request to implement this change? Yes

[kaicataldo]

I’ll champion this. While this would be a change of scope, I agree that this is in the spirit of the rule and is useful. I don’t think we should create another rule for this.

That being said, I think this has to go behind an option that is turned off by default and that we should make it the default behavior in a future major release.

[platinumazure]

This is now accepted.