Change any to unknown

[keeganwitt]

A follow-up to this conversation. We need the second argument to be an array, then we can use unknown instead of any, and this is the best practice.

After starting this change, I realized there were multiple places we could change an any to unknown.

[codecov]

Codecov Report

Merging #477 (d97afc9) into main (552e950) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              main      #477   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           71        71           
  Lines          582       582           
  Branches       148       148           
=========================================
  Hits           582       582           

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 552e950...d97afc9. Read the comment docs.

[SimenB]

Remove this as well?

jest-extended/.eslintrc

Lines 55 to 56 in 801c068

	// should probably enable this, but it's a bit too strict for now
	"@typescript-eslint/no-explicit-any": "off"

[keeganwitt]

Remove this as well?

jest-extended/.eslintrc

Lines 55 to 56 in 801c068

	// should probably enable this, but it's a bit too strict for now
	"@typescript-eslint/no-explicit-any": "off"

There are 71 any usages left still in index.d.ts. However, there aren't any elsewhere, so maybe it'd still be a better default and turn off the rule just for this 1 file.

[SimenB]

Only ts-file we've got atm, so that makes sense