Skip to content

Add mypy and pyright checks - #15

Merged
Radhika Gupta (rads-1996) merged 2 commits into
microsoft:mainfrom
rads-1996:add-mypy-and-pyright
Apr 6, 2026
Merged

Radhika Gupta (rads-1996) merged 2 commits into
microsoft:mainfrom
rads-1996:add-mypy-and-pyright

Conversation

@rads-1996

Copy link
Copy Markdown
Member

No description provided.

Copilot AI review requested due to automatic review settings April 3, 2026 20:02

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds static type-checking tool dependencies and configuration to the repository’s Python project settings, preparing the codebase to run mypy and pyright alongside existing quality checks.

Changes:

  • Add mypy and pyright to the dev optional dependencies.
  • Introduce [tool.mypy] configuration (Python 3.10, stricter typing options).
  • Introduce [tool.pyright] configuration (Python 3.10, basic type checking mode).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread pyproject.toml
@rads-1996
Radhika Gupta (rads-1996) marked this pull request as draft April 3, 2026 20:05

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/pr-validation.yml
@rads-1996
Radhika Gupta (rads-1996) merged commit 4c25a58 into microsoft:main Apr 6, 2026
10 checks passed
@rads-1996
Radhika Gupta (rads-1996) deleted the add-mypy-and-pyright branch April 6, 2026 13:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants