Narrow VersionValue to omit true values

[ddbeck]

Summary

BCD's published types don't match the JSON Schema; this PR partially fixes this.

Test results and supporting details

VersionValue has the type string | boolean | null; it should have the type string | false.

Unfortunately, setting VesionValue to string | false causes several type errors in linter code. And it's not just dead code either; replacing places true or null returns into thrown exceptions causes many errors.

Related issues

• feat(schema)!: remove null and true version values #24174 (comment)

[github-actions]

Tip: Review these changes grouped by change (recommended for most PRs), or grouped by feature (for large PRs).

[Elchi3]

Seems like #27428 is a PR that goes the extra miles and should be reviewed instead.

[ddbeck]

I added "closes" text to the description of the other PR. If/when it merges, this one ought to be automatically closed.

[queengooborg]

Actually, why don't we just close this PR now and focus on reviewing the other PR?

[ddbeck]

I didn't close this one because it's where the actual bug is reported. It's notionally a PR but it's also the issue for the bug itself. It's not fixed and isn't a wontfix (presumably), so I figured "closes" was the best way to tell the history of the changes here (and for me personally, this PR being resolved closed would be a signal to start deleting the code that this bug is a blocker on).