cli exits with code 0 when `reveal_type`/`reveal_locals` are used

[DetachHead]

reveal_type(1)

> basedpyright .\a.py
a.py
  a.py:3:13 - information: Type of "1" is "Literal[1]"
0 errors, 0 warnings, 1 note
> echo $LASTEXITCODE 
0

you would never want these to be committed, so it should fail the CI.

in mypy, all notes cause exit code 1. idk if we should do the same or not. are there other notes where it shouldn't fail? if so, maybe reveal_type and reveal_locals should be an error instead of a note

[DetachHead]

also worth noting that it will cause a runtime error unless you import it from typing (pyright special cases it to not report an error when it's not imported), or if you use the expected_text argument, which appears to be a fake argument used by pyright for its own tests

[DetachHead]

i think instead we should just remove the special-casing on reveal_type that prevents reportUndefinedVariable from being reported on it