Rule Change: make no-return-await suggestable

What rule do you want to change?

no-return-await

What change to do you want to make?

Implement autofix

How do you think the change should be implemented?

A new option

Example code

async function foo {
  return await Promise.resolve("bar")
}

What does the rule currently do for this code?

// Error, but don’t change: async function foo { return await Promise.resolve(“bar”) }

What will the rule do after it’s changed?

// Fix it async function foo { return Promise.resolve(“bar”) }

Participation

• I am willing to submit a pull request to implement this change.

Additional comments

Hi! There have been discussions about whether no-return-await should be a core rule. These are resolved - and, consequently, I would argue that the rule should be auto-fixable.

[mdjermanovic]

I agree with @yeonjuan, per https://github.com/eslint/eslint/issues/7593#issuecomment-260419426 this doesn’t seem safe for autofixing.

Perhaps we could consider implementing this as suggestions?

[dbartholomae]

I’ve opened #16637 to fix this.