Add pyright type-checking

[tjkuson]

Hello, I was working with this plugin and encountered this issue.

I added pyright as a tox test matching the mypy hook.

This reveals one error involving function.parent possibly being None. I resolved this by adding an assertion, as that appears to be most consistent with the rest of the codebase.

Closes #731

[tjkuson]

I'll close this for now, there's an issue with pyright and pre-commit CI: RobertCraigie/pyright-python#345

[seifertm]

Rather than running pyright in pre-commit, we can consider running it in a dedicated tox environment. This would give us the benefit of having a pyright check for the code base, while 90% of typing issues should still be caught by the mypy pre-commit hook.

@tjkuson Do you think this approach would side-step the issue?

[codecov-commenter]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 88.88%. Comparing base (07cb01f) to head (5ad6dab).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files

@@ Coverage Diff @@ ## main #1172 +/- ## ========================================== + Coverage 88.16% 88.88% +0.72%  ========================================== Files 2 2 Lines 397 405 +8 Branches 44 44 ========================================== + Hits 350 360 +10  Misses 35 35 + Partials 12 10 -2 

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[tjkuson]

@seifertm I've moved the pyright check from pre-commit to tox and re-opened the PR.

[seifertm]

Thanks! This will help catch issues that are flagged by pyright, but not by mypy (even though we don't run pyright in a pre-commit hook).