Fix yarn check-readme

[nflaig]

Maybe we should run yarn check-readme in CI along with tests to keep these readmes compile-able?

We actually do

lodestar/.github/workflows/test.yml

Lines 144 to 145 in 88744d8

	- name: README check
	run: yarn check-readme

But it doesn't seem to care at all

~/projects/ethereum/lodestar/packages/light-client [unstable ≡ +0 ~1 -0 !]> yarn check-readme
yarn run v1.22.19
$ typescript-docs-verifier
ℹ Compiling documentation TypeScript code snippets from README.md
ℹ Found 1 TypeScript snippets
✔ All snippets compiled OK

@jeluard how did you verify and determine the changes to make it compile, just copy pasted it into a .ts file?

Originally posted by @nflaig in #6297 (comment)

[jeluard]

@nflaig yes just a plain copy

[jeluard]

Some findings:

• it actually detects some things, like unknown variables at top-level, but not inside a function
• if I comment "transpileOnly": true, all errors seem to be picked up

[matthewkeil]

Nice catch!!! Your thoroughness is commendable @jeluard!!! It looks like we can pass a “—project” to the typescript-docs-verifier. Perhaps we have a tsconfig.docs.json with that set to false to make sure it runs correctly

[philknows]

@nflaig Any updates on your PR #6868 ?

[nflaig]

@nflaig Any updates on your PR #6868 ?

hmm there was no response on bbc/typescript-docs-verifier#31 and the module we use seems unmaintained, maybe we just fork it? I haven't found an alternative