Repository navigation
Add support for local mypy, pylint, black checks - #14
Merged
Radhika Gupta (rads-1996) merged 7 commits intoApr 6, 2026
Merged
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
Adds a self-contained eng/tox configuration to enable running local static checks (mypy, pylint, black) via tox from the repo root.
Changes:
- Introduces
eng/tox/tox.iniwith environments formypy,pylint,black, andpytest. - Adds helper runner scripts for mypy/pylint and a tox python hook to provide a
{repository_root}constant. - Adds
dev_requirements.txtused by tox env dependency installation.
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 7 comments.
Show a summary per file
| File | Description |
|---|---|
| eng/tox/toxfile.py | Adds a tox hook that derives and exposes {repository_root}. |
| eng/tox/tox.ini | Defines tox envs/commands for mypy, pylint, black, pytest. |
| eng/tox/run_pylint.py | Script wrapper to run pylint on src/ and optionally tests/. |
| eng/tox/run_mypy.py | Script wrapper to run mypy on src/ and optionally tests/ (samples currently disabled). |
| dev_requirements.txt | Shared dependency list referenced by tox envs. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Contributor
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 5 out of 5 changed files in this pull request and generated 4 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Radhika Gupta (rads-1996)
force-pushed
the
add-tox-checks
branch
from
April 6, 2026 17:36
f7ea7c3 to
104ca86
Compare
Hector Hernandez (hectorhdzg)
approved these changes
Apr 6, 2026
Radhika Gupta (rads-1996)
requested review from
Jackson Weber (JacksonWeber),
Nev (MSNev) and
Leighton Chen (lzchen)
as code owners
April 6, 2026 19:59
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Use the following commands to run these checks locally -
python -m tox run -e pylint -c <pathto/eng/tox/tox.ini> --root .python -m tox run -e mypy -c <pathto/eng/tox/tox.ini> --root .python -m tox run -e black -c <pathto/eng/tox/tox.ini> --root .