Drop the Google TypeScript linter module gts, instead sticking with the base plugin:@typescript-eslint/recommended and plugin:@typescript-eslint/recommended-requiring-type-checking configuration.
Tweak the linter options to be a little stronger in places:
Require await.
Do not allow floating promises.
Do not allow unsafe call or return.
Do not allow unused vars.
Add back some linter options, presumably from gts, that we need:
@typescript-eslint/no-non-null-assertion": "off"
@typescript-eslint/no-var-requires": "off"
Reformat all source files with the new lint settings, and fix a few minor issues - mostly around removing floating promises.
Fix some spelling typos.
In principle, this PR should be mostly aesthetic and make no real changes to behaviour. At most, there are some promises that are now awaited that were not before.
Drop the Google TypeScript linter module
gts
, instead sticking with the baseplugin:@typescript-eslint/recommended
andplugin:@typescript-eslint/recommended-requiring-type-checking
configuration.Tweak the linter options to be a little stronger in places:
await
.Add back some linter options, presumably from
gts
, that we need:@typescript-eslint/no-non-null-assertion": "off"
@typescript-eslint/no-var-requires": "off"
Reformat all source files with the new lint settings, and fix a few minor issues - mostly around removing floating promises.
Fix some spelling typos.
In principle, this PR should be mostly aesthetic and make no real changes to behaviour. At most, there are some promises that are now awaited that were not before.