feat(eslint-config): enable "no-return-await" rule

See https://eslint.org/docs/rules/no-return-await:

> Since the return value of an async function is always wrapped in
> `Promise.resolve, `return await` doesn’t actually do anything except
> add extra time before the overarching `Promise` resolves or rejects.

BREAKING CHANGE: "return await" is no longer allowed, just return the
promise without awaiting its resolution.

Author: bajtos

packages/eslint-config/eslintrc.js

@@ -56,6 +56,10 @@ module.exports = {
     // TypeScript allows the same name for namespace and function
     'no-redeclare': 'off',
 
+    // Avoid promise rewrapping
+    // https://exploringjs.com/es2016-es2017/ch_async-functions.html#_returned-promises-are-not-wrapped
+    'no-return-await': 'error',
+
     /**
      * Rules imported from eslint-config-loopback
      */