You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
More often than not, we want to await and not doing so leads to scary bugs. For cases where we explicitly don't want to await, we can add a lint ignore line comment.
running pnpm lint in CLI shows this (and does catch some await errors)
/Users/kevin/Projects/latticexyz/mud/packages/cli/dist/index.d.ts
0:0 error Parsing error: ESLint was configured to run on `<tsconfigRootDir>/dist/index.d.ts` using `parserOptions.project`: /users/kevin/projects/latticexyz/mud/tsconfig.json
However, that TSConfig does not include this file. Either:
- Change ESLint's list of included files to not include this file
- Change that TSConfig to include this file
- Create a new TSConfig that includes this file and include it in your parserOptions.project
See the typescript-eslint docs for more info: https://typescript-eslint.io/linting/troubleshooting#i-get-errors-telling-me-eslint-was-configured-to-run--however-that-tsconfig-does-not--none-of-those-tsconfigs-include-this-file
but it doesn't seem to be catching the issue in #3119
More often than not, we want to
await
and not doing so leads to scary bugs. For cases where we explicitly don't want toawait
, we can add a lint ignore line comment.https://github.com/typescript-eslint/typescript-eslint/blob/main/packages/eslint-plugin/docs/rules/no-floating-promises.md
The text was updated successfully, but these errors were encountered: