Update: Disable require-await for async generators (fixes #12459)

[dmitryrn]

What is the purpose of this pull request? (put an "X" next to item)

[X] Changes an existing rule

What changes did you make? (Give an overview)
I disabled require-await for async generators. #12459

Is there anything you'd like reviewers to focus on?
Should documentation of require-await updated somehow?

[jsf-clabot]

All committers have signed the CLA.

[mdjermanovic]

Thanks for the PR! Labeling as accepted because the related issue is accepted.

Should documentation of require-await updated somehow?

I think it would be good to update the documentation, maybe just with a short note that the rule doesn't check async generators.

Would also like to doublecheck with the team:

• Is it accepted to skip all async generators or just those with yield *? (and if all, it would be nice to add a test without yield *).
• Update: looks appropriate, although this change produces fewer errors?

[kaicataldo]

@dmitryrn Friendly ping

[kaicataldo]

One more friendly ping

[mdjermanovic]

Assuming it was accepted to ignore all async generators regardless of any other conditions, the change in the code looks good!

It would be nice to add just two small details:

• A valid test case with an async generator that doesn't have yield * (and isn't empty and doesn't have await).
• A sentence in the documentation saying that this rule doesn't check async generators.

[kaicataldo]

@dmitryrn Friendly ping. Are you willing to continue working on this?

[kaicataldo]

Closing this in favor of #13048 since we haven't heard back from the PR author.