Update all non-major dependencies

[renovate]

This PR contains the following updates:

Package	Change	Age	Adoption	Passing	Confidence
@types/chai	4.2.15 -> 4.2.18
@types/chai-as-promised	7.1.3 -> 7.1.4
@types/mocha	8.2.1 -> 8.2.2
@typescript-eslint/eslint-plugin	4.17.0 -> 4.24.0
@typescript-eslint/parser	4.17.0 -> 4.24.0
cockroachdb/cockroach	v20.2.5 -> v20.2.10
dotenv	8.2.0 -> 8.6.0
eslint (source)	7.22.0 -> 7.27.0
eslint-plugin-import	2.22.1 -> 2.23.3
eslint-plugin-prettier	3.3.1 -> 3.4.0
mocha (source)	8.3.2 -> 8.4.0
pg	8.5.1 -> 8.6.0
postgres	13.2-alpine -> 13.3-alpine
postgres	12.6-alpine -> 12.7-alpine
postgres	11.11-alpine -> 11.12-alpine
postgres	10.16-alpine -> 10.17-alpine
prettier (source)	2.2.1 -> 2.3.0
sinon-chai	3.5.0 -> 3.6.0
typescript (source)	4.2.3 -> 4.2.4

---

Release Notes

typescript-eslint/typescript-eslint

v4.24.0

Compare Source

Bug Fixes

• eslint-plugin: [no-shadow] fix static class method generics shadowing class generics (#​3393) (b1e1c8a), closes #​2592
• eslint-plugin: [no-unsafe-*] special case handling for the empty map constructor with no generics (#​3394) (cae4f4a), closes #​2109

Features

• eslint-plugin: [dot-notation] optionally allow square bracket notation where an index signature exists in conjunction with noPropertyAccessFromIndexSignature (#​3361) (37ec2c2)

v4.23.0

Compare Source

Bug Fixes

• scope-manager: fix visiting TSAsExpression in assignment (#​3355) (87521a0)

Features

• experimental-utils: Include getCwd() in RuleContext type (#​3308) (2b75c11)
• refactor to split AST specification out as its own module (#​2911) (25ea953)

4.22.1 (2021-05-04)

Note: Version bump only for package @​typescript-eslint/typescript-eslint

v4.22.1

Compare Source

Note: Version bump only for package @​typescript-eslint/typescript-eslint

v4.22.0

Compare Source

Bug Fixes

• eslint-plugin: [no-unsafe-argument] handle tuple types on rest arguments (#​3269) (6f8cfe6)

Features

• eslint-plugin: [prefer-regexp-exec] add autofix (#​3207) (e2cbeef)

v4.21.0

Compare Source

Bug Fixes

• eslint-plugin: [no-type-alias] consider keyof as an alias (#​3242) (329ef02)
• eslint-plugin: [no-unnecessary-type-assertion] correct bad fix for angle bracket assertion (#​3244) (265a039)
• eslint-plugin: [restrict-plus-operands] consider template literal types as strings (#​3234) (ccfd68e)
• eslint-plugin: [strict-boolean-expressions] account for truthy literals (#​3236) (0913f40)
• eslint-plugin: always ignore assignments in no-unnecessary-type-assertion (#​3235) (0221476)

Features

• eslint-plugin: [no-unsafe-argument] add rule (#​3256) (b1aa7dc), closes #​791
• eslint-plugin: [no-unsafe-call][no-unsafe-member-access] improve report messages for this for noImplicitThis (#​3199) (b1b26c4)

v4.20.0

Compare Source

Features

• eslint-plugin: [space-infix-ops] support for class properties and type aliases (#​3231) (5414bf2)
• eslint-plugin: [type-annotation-spacing] handle space between ? and : (#​3138) (40bdb0b)

v4.19.0

Compare Source

Bug Fixes

• eslint-plugin: [member-delimiter-style] correct invalid fix for multiline with params on the same line (#​3177) (7ad343b)
• eslint-plugin: [promise-function-async] bad fixer with computed and literal methods (#​3163) (e3a3ea0)
• typescript-estree: [ts 4.2] add support for import type equal declaration (#​3189) (6a25faf)

Features

• eslint-plugin: [object-curly-spacing] support MappedType (#​3176) (0557a43)
• eslint-plugin: [unbound-method] improve error message (#​3203) (5cc5d2e), closes #​3201

v4.18.0

Compare Source

Bug Fixes

• eslint-plugin: [explicit-module-boundary-types] fixes #​2864 related to functions in nested object properties (#​3178) (55e1fba)
• eslint-plugin: [no-extran-class] allowWithDecorator should ignore other errors (#​3160) (a148673)

Features

• eslint-plugin: add package type declaration (#​3164) (08b058a)

motdotla/dotenv

v8.6.0

Compare Source

Added

• define package.json in exports

v8.5.1

Compare Source

Changed

• updated dev dependencies via npm audit

v8.5.0

Compare Source

Added

• allow for import "dotenv/config"

v8.4.0

Compare Source

Changed

• point to exact types file to work with VS Code

v8.3.0

Compare Source

Changed

• Breaking: drop support for Node v8 (mistake to be released as minor bump. later bumped to 9.0.0. see above.)

eslint/eslint

v7.27.0

Compare Source

• 2c0868c Chore: merge all html formatter files into html.js (#​14612) (Milos Djermanovic)
• 9e9b5e0 Update: no-unused-vars false negative with comma operator (fixes #​14325) (#​14354) (Nitin Kumar)
• afe9569 Chore: use includes instead of indexOf (#​14607) (Mikhail Bodrov)
• c0f418e Chore: Remove lodash (#​14287) (Stephen Wade)
• 52655dd Update: no-restricted-imports custom message for patterns (fixes #​11843) (#​14580) (Alex Holden)
• 967b1c4 Chore: Fix typo in large.js (#​14589) (Ikko Ashimine)
• 2466a05 Sponsors: Sync README with website (ESLint Jenkins)
• fe29f18 Sponsors: Sync README with website (ESLint Jenkins)
• 086c1d6 Chore: add more test cases for no-sequences (#​14579) (Nitin Kumar)
• 6a2ced8 Docs: Update README team and sponsors (ESLint Jenkins)

v7.26.0

Compare Source

• aaf65e6 Upgrade: eslintrc for ModuleResolver fix (#​14577) (Brandon Mills)
• ae6dbd1 Fix: track variables, not names in require-atomic-updates (fixes #​14208) (#​14282) (Patrick Ahmetovic)
• 6a86e50 Chore: remove loose-parser tests (fixes #​14315) (#​14569) (Milos Djermanovic)
• ee3a3ea Fix: create .eslintrc.cjs for module type (#​14304) (Nitin Kumar)
• 6791dec Docs: fix example for require-atomic-updates (#​14562) (Milos Djermanovic)
• 388eb7e Sponsors: Sync README with website (ESLint Jenkins)
• f071d1e Update: Add automated suggestion to radix rule for parsing decimals (#​14291) (Bryan Mishkin)
• 0b6a3f3 New: Include XO style guide in eslint --init (#​14193) (Federico Brigante)

v7.25.0

Compare Source

• 5df5e4a Update: highlight last write reference for no-unused-vars (fixes #​14324) (#​14335) (Nitin Kumar)
• 0023872 Docs: Add deprecated note to working-with-rules-deprecated page (#​14344) (Michael Novotny)
• 36fca70 Chore: Upgrade eslump to 3.0.0 (#​14350) (Stephen Wade)
• 59b689a Chore: add node v16 (#​14355) (薛定谔的猫)
• 35a1f5e Sponsors: Sync README with website (ESLint Jenkins)
• fb0a92b Chore: rename misspelled identifier in test (#​14346) (Tobias Nießen)
• f2babb1 Docs: update pull request template (#​14336) (Nitin Kumar)
• 02dde29 Docs: Fix anchor in 'docs/developer-guide/working-with-rules.md' (#​14332) (Nate-Wilkins)
• 07d14c3 Chore: remove extraneous command from lint-staged config (#​14314) (James George)
• 41b3570 Update: lint code block with same extension but different content (#​14227) (JounQin)
• eb29996 Docs: add more examples with arrow functions for no-sequences rule (#​14313) (Nitin Kumar)

v7.24.0

Compare Source

• 0c346c8 Chore: ignore pnpm-lock.yaml (#​14303) (Nitin Kumar)
• f06ecdf Update: Add disallowTemplateShorthand option in no-implicit-coercion (#​13579) (Remco Haszing)
• 71a80e3 Docs: fix broken links in Node.js API docs toc (#​14296) (u-sho (Shouhei Uechi))
• bd46dc4 Docs: Fix incorrect reference to "braces" in arrow-parens (#​14300) (emclain)
• 0d6235e Docs: update header in max-lines (#​14273) (Shinigami)
• 70c9216 Docs: Update issue triage to include blocked column (#​14275) (Nicholas C. Zakas)
• abca186 Docs: Fix typo in suggestions section (#​14293) (Kevin Partington)
• c4d8b0d Fix: no-unused-vars ignoreRestSiblings check assignments (fixes #​14163) (#​14264) (YeonJuan)
• b51d077 Update: add ignoreNonDeclaration to no-multi-assign rule (fixes #​12545) (#​14185) (t-mangoe)
• c981fb1 Chore: Upgrade mocha to 8.3.2 (#​14278) (Stephen Wade)
• 147fc04 Docs: Fix repro:needed label in bug report template (#​14285) (Milos Djermanovic)
• e1cfde9 Docs: Update bug report template (#​14276) (Nicholas C. Zakas)
• c85c2f1 Docs: Add fatal to Node.js API LintMessage type (#​14251) (Brandon Mills)

v7.23.0

Compare Source

• 687ccae Update: add option "allowInParentheses" to no-sequences (fixes #​14197) (#​14199) (Daniel Rentz)
• dbf2529 Sponsors: Sync README with website (ESLint Jenkins)
• 4bdf2c1 Sponsors: Sync README with website (ESLint Jenkins)
• 49d1697 Chore: Upgrade eslint-plugin-jsdoc to v25 and remove --legacy-peer-deps (#​14244) (Brandon Mills)
• 43f1685 Update: --quiet should not supress --max-warnings (fixes #​14202) (#​14242) (Milos Djermanovic)
• 909c727 Docs: Add valid example that shows vars in a block scope (#​14230) (Ed S)
• 28583eb Fix: no-mixed-operators false positives with ? : (fixes #​14223) (#​14226) (Milos Djermanovic)
• a99eb2d Fix: Clarify line breaks in object-curly-newline (fixes #​14024) (#​14063) (armin yahya)
• 8984c91 Update: eslint --env-info output os info (#​14059) (薛定谔的猫)
• 2a79306 Sponsors: Sync README with website (ESLint Jenkins)
• ebd7026 Docs: Fix typo (#​14225) (Greg Finley)
• a2013fc Sponsors: Sync README with website (ESLint Jenkins)

benmosher/eslint-plugin-import

v2.23.3

Compare Source

Fixed

• [no-restricted-paths]: fix false positive matches ([#​2090], thanks [@​malykhinvi])
• [no-cycle]: ignore imports where imported file only imports types of importing file ([#​2083], thanks [@​cherryblossom000])
• [no-cycle]: fix false negative when file imports a type after importing a value in Flow ([#​2083], thanks [@​cherryblossom000])
• [order]: restore default behavior unless type is in groups ([#​2087], thanks [@​grit96])

Changed

• [Docs] Add no-relative-packages to list of to the list of rules ([#​2075], thanks [@​arvigeus])

v2.23.2

Compare Source

Changed

• [meta] add safe-publish-latest; use prepublishOnly script for npm 7+

v2.23.1

Compare Source

Fixed

• [newline-after-import]: fix crash with export {} syntax ([#​2063], [#​2056], thanks [@​ljharb])
• ExportMap: do not crash when tsconfig lacks .compilerOptions ([#​2067], thanks [@​ljharb])
• [order]: fix alphabetical sorting ([#​2071], thanks [@​grit96])

v2.23.0

Compare Source

Added

• [no-commonjs]: Also detect require calls with expressionless template literals: require(`x`) ([#​1958], thanks [@​FloEdelmann])
• [no-internal-modules]: Add forbid option ([#​1846], thanks [@​guillaumewuip])
• add [no-relative-packages] ([#​1860], [#​966], thanks [@​tapayne88] [@​panrafal])
• add [no-import-module-exports] rule: report import declarations with CommonJS exports ([#​804], thanks [@​kentcdodds] and [@​ttmarek])
• [no-unused-modules]: Support destructuring assignment for export. ([#​1997], thanks [@​s-h-a-d-o-w])
• [order]: support type imports ([#​2021], thanks [@​grit96])
• [order]: Add warnOnUnassignedImports option to enable warnings for out of order unassigned imports ([#​1990], thanks [@​hayes])

Fixed

• [export]/TypeScript: properly detect export specifiers as children of a TS module block ([#​1889], thanks [@​andreubotella])
• [order]: ignore non-module-level requires ([#​1940], thanks [@​golopot])
• [no-webpack-loader-syntax]/TypeScript: avoid crash on missing name ([#​1947], thanks [@​leonardodino])
• [no-extraneous-dependencies]: Add package.json cache ([#​1948], thanks [@​fa93hws])
• [prefer-default-export]: handle empty array destructuring ([#​1965], thanks [@​ljharb])
• [no-unused-modules]: make type imports mark a module as used (fixes #​1924) ([#​1974], thanks [@​cherryblossom000])
• [no-cycle]: fix perf regression ([#​1944], thanks [@​Blasz])
• [first]: fix handling of import = require ([#​1963], thanks [@​MatthiasKunnen])
• [no-cycle]/[extensions]: fix isExternalModule usage ([#​1696], thanks [@​paztis])
• [extensions]/[no-cycle]/[no-extraneous-dependencies]: Correct module real path resolution ([#​1696], thanks [@​paztis])
• [no-named-default]: ignore Flow import type and typeof ([#​1983], thanks [@​christianvuerings])
• [no-extraneous-dependencies]: Exclude flow typeof imports ([#​1534], thanks [@​devongovett])
• [newline-after-import]: respect decorator annotations ([#​1985], thanks [@​lilling])
• [no-restricted-paths]: enhance performance for zones with except paths ([#​2022], thanks [@​malykhinvi])
• [no-unresolved]: check import() ([#​2026], thanks [@​aladdin-add])

Changed

• [Generic Import Callback] Make callback for all imports once in rules ([#​1237], thanks [@​ljqx])
• [Docs] [no-named-as-default]: add semicolon ([#​1897], thanks [@​bicstone])
• [Docs] no-extraneous-dependencies: correct peerDependencies option default to true ([#​1993], thanks [@​dwardu])
• [Docs] order: Document options required to match ordering example ([#​1992], thanks [@​silviogutierrez])
• [Tests] no-unresolved: add tests for import() ([#​2012], thanks [@​davidbonnet])
• [Docs] Add import/recommended ruleset to README ([#​2034], thanks [@​edemaine])

prettier/eslint-plugin-prettier

v3.4.0

Compare Source

• feat: support processor virtual filename (#​401) (ee0ccc6)
• Simplify report logic (#​380) (d993f24)
• Update: README.md (#​375) (3ea4242)

mochajs/mocha

v8.4.0

Compare Source

🎉 Enhancements

• #​4502: CLI file parsing errors now have error codes (@​evaline-ju)

🐛 Fixes

• #​4614: Watch: fix crash when reloading files (@​outsideris)

📖 Documentation

• #​4630: Add options.require to Mocha constructor for root hook plugins on parallel runs (@​juergba)
• #​4617: Dynamically generating tests with top-level await and ESM test files (@​juergba)
• #​4608: Update default file extensions (@​outsideris)

Also thanks to @​outsideris for various improvements on our GH actions workflows.

brianc/node-postgres

v8.6.0

Compare Source

• Better SASL error messages & more validation on bad configuration.
• Export DatabaseError.
• Add ParameterDescription support to protocol parsing.
• Fix typescript typedefs with --isolatedModules.

prettier/prettier

v2.3.0

Compare Source

diff

🔗 Release Notes

domenic/sinon-chai

v3.6.0

Compare Source

• Add support for sinon 10 (#​149)

Microsoft/TypeScript

v4.2.4

Compare Source

For release notes, check out the release announcement.

For the complete list of fixed issues, check out the

• fixed issues query for TypeScript v4.2.0 (Beta).
• fixed issues query for TypeScript v4.2.1 (RC).
• fixed issues query for TypeScript v4.2.2 (Stable).
• fixed issues query for TypeScript v4.2.3.
• fixed issues query for TypeScript v4.2.4.

Downloads are available on:

• npm
• Visual Studio 2017/2019 (Select new version in project options)
• NuGet package

---

Configuration

📅 Schedule: "before 3am on Monday" (UTC).

🚦 Automerge: Disabled due to failing status checks.

♻️ Rebasing: Renovate will not automatically rebase this PR, because other commits have been found.

👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

---

• If you want to rebase/retry this PR, check this box.

---

This PR has been generated by WhiteSource Renovate. View repository job log here.

[littlewhywhat]

@Shinigami92 unfortunately lint is broken:

error logs

#!/bin/bash -eo pipefail
npm run lint

> node-pg-migrate@5.9.0 lint /home/circleci/project
> eslint . bin/* --ext .js,.ts


/home/circleci/project/mocha.bootstrap.js
  6:24  error  Unexpected use of file extension "json" for "./tsconfig-test.json"  import/extensions

/home/circleci/project/src/migration-builder.ts
  211:17  error  Replace `·<T·extends·Operation>(operation:·T)·=>` with `⏎······<T·extends·Operation>(operation:·T)·=>⏎·····`                                prettier/prettier
  212:7   error  Insert `··`                                                                                                                                 prettier/prettier
  213:1   error  Insert `··`                                                                                                                                 prettier/prettier
  214:11  error  Insert `··`                                                                                                                                 prettier/prettier
  215:1   error  Insert `··`                                                                                                                                 prettier/prettier
  216:9   error  Insert `··`                                                                                                                                 prettier/prettier
  217:1   error  Insert `··`                                                                                                                                 prettier/prettier
  218:7   error  Insert `··`                                                                                                                                 prettier/prettier
  219:1   error  Insert `··`                                                                                                                                 prettier/prettier
  220:7   error  Insert `··`                                                                                                                                 prettier/prettier
  221:1   error  Insert `··`                                                                                                                                 prettier/prettier
  333:19  error  Replace `·<T·extends·any[],·R>(operation:·(...args:·T)·=>·R)·=>` with `⏎······<T·extends·any[],·R>(operation:·(...args:·T)·=>·R)·=>⏎·····`  prettier/prettier
  334:1   error  Insert `··`                                                                                                                                 prettier/prettier
  335:1   error  Insert `··`                                                                                                                                 prettier/prettier
  336:1   error  Replace `······` with `········`                                                                                                            prettier/prettier
  337:1   error  Insert `··`                                                                                                                                 prettier/prettier
  338:1   error  Insert `··`                                                                                                                                 prettier/prettier

/home/circleci/project/src/migration.ts
  10:1  error  '@types/mkdirp' should be listed in the project's dependencies, not devDependencies  import/no-extraneous-dependencies

/home/circleci/project/src/operations/operators.ts
  103:20  error  Replace `·(mOptions:·MigrationOptions)·=>` with `⏎··(mOptions:·MigrationOptions)·=>⏎·`  prettier/prettier
  104:3   error  Insert `··`                                                                             prettier/prettier
  105:1   error  Insert `··`                                                                             prettier/prettier
  106:5   error  Insert `··`                                                                             prettier/prettier
  107:1   error  Insert `··`                                                                             prettier/prettier
  108:5   error  Insert `··`                                                                             prettier/prettier
  109:1   error  Insert `··`                                                                             prettier/prettier
  111:5   error  Insert `··`                                                                             prettier/prettier
  112:1   error  Insert `··`                                                                             prettier/prettier
  114:3   error  Insert `··`                                                                             prettier/prettier
  115:1   error  Insert `··`                                                                             prettier/prettier
  116:5   error  Insert `··`                                                                             prettier/prettier
  117:3   error  Insert `··`                                                                             prettier/prettier
  119:1   error  Insert `··`                                                                             prettier/prettier
  120:1   error  Insert `··`                                                                             prettier/prettier

/home/circleci/project/src/operations/tables.ts
  271:12  error  Replace `·columns:·columnLines,·constraints:·crossColumnConstraints,·comments:·columnComments·=·[]·}·=·parseColumns(⏎······tableName,⏎······columns,⏎······mOptions,⏎····` with `⏎······columns:·columnLines,⏎······constraints:·crossColumnConstraints,⏎······comments:·columnComments·=·[],⏎····}·=·parseColumns(tableName,·columns,·mOptions`  prettier/prettier

/home/circleci/project/src/operations/views.ts
   8:22  error  Replace `·<T·extends·Nullable<ViewOptions>,·K·extends·keyof·T>(options:·T)·=>` with `⏎··<T·extends·Nullable<ViewOptions>,·K·extends·keyof·T>(options:·T)·=>⏎·`  prettier/prettier
   9:1   error  Replace `··` with `····`                                                                                                                                        prettier/prettier
  10:3   error  Insert `··`                                                                                                                                                     prettier/prettier
  11:1   error  Insert `··`                                                                                                                                                     prettier/prettier

/home/circleci/project/src/operations/viewsMaterialized.ts
  24:28  error  Replace `·<T·extends·Nullable<StorageParameters>,·K·extends·keyof·T>(storageParameters:·T)·=>·(` with `⏎··<T·extends·Nullable<StorageParameters>,·K·extends·keyof·T>(storageParameters:·T)·=>`  prettier/prettier
  25:3   error  Replace `key:·K,⏎` with `(key:·K`                                                                                                                                                               prettier/prettier
  27:3   error  Insert `··`                                                                                                                                                                                     prettier/prettier
  28:3   error  Insert `··`                                                                                                                                                                                     prettier/prettier
  29:1   error  Insert `··`                                                                                                                                                                                     prettier/prettier

/home/circleci/project/src/runner.ts
  121:20  error  Insert `⏎······`  prettier/prettier
  122:1   error  Insert `··`       prettier/prettier
  123:1   error  Insert `··`       prettier/prettier

✖ 47 problems (47 errors, 0 warnings)
  45 errors and 0 warnings potentially fixable with the `--fix` option.

npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR! node-pg-migrate@5.9.0 lint: `eslint . bin/* --ext .js,.ts`
npm ERR! Exit status 1
npm ERR! 
npm ERR! Failed at the node-pg-migrate@5.9.0 lint script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR! A complete log of this run can be found in:
npm ERR!     /home/circleci/.npm/_logs/2021-05-15T18_00_04_946Z-debug.log


Exited with code exit status 1
CircleCI received exit code 1

also smth weird with db start for test-pg-13 step but let's do one step at a time. Not sure if this PR is still relevant though (there are a few others with dep updates).

[Shinigami92]

@littlewhywhat How do you want me to fix this? 🙂
I can provide multiple solutions:

1. I jump into this auto-generated branch and try to fix everything in place
2. I create manual PRs for every dependency update and you can check them one by one
3. I create a collection of dependency updates and another PR that handles dependencies with breakings
4. I create a whole PR for everything (like 1) but do everything at once on my own manually

[littlewhywhat]

@Shinigami92 FYI I filed this ticket #787.

I think let's let renovate-bot to do its job. I love that it provides all breaking changes info that is needed. So from your option probably it will be fixing the PR created by the bot here in place (without changing proposed versions).

Otherwise, what about we review + merge all successful PRs first? I merged one and gonna try to find time to go through others... you could try to fix this one in this PR by rolling smth back down or understanding the failure reason. Many thanks 🙏

P.S. I created project Maintenance (https://github.com/salsita/node-pg-migrate/projects/1) where in Priority column I tried to sort all stuff we have pending by the priority. So if you don't feel like working on this PR - feel free to assign to me and check, choose something from the top 👍

[Shinigami92]

Okay, so I will choose open 1. and try it today

[Shinigami92]

We are affected by import-js/eslint-plugin-import#2078
So I will downgrade to 2.22.1 for now

[Shinigami92]

This is just a workaround for now, cause when I move line eslint-disable-next-line just one line down, the formatting breaks completely
This part is already ugly ^^ IMO we could refactor it to get rid of this inline generic function later in another PR or so

[littlewhywhat]

or we should use some alternative to those strange (to my taste) prettier formatting rules :D

[Shinigami92]

I think the only rule change that could help here would be increment the printWidth. But this is already on 120 and this should not increased to a higher value.
So what alternative did you think of? Not using prettier would not be an option to me, cause I use it in all my projects, it has a wide eco-system and it's the best formatting tool out there with a strong team behind.

[Shinigami92]

@littlewhywhat Everything green 🎉
No direct code changes, just some formatting and linting

[littlewhywhat]

Awesome work! 🚀 thank you!