Skip to content

Remove unused mypy dependency - #124

Merged
hzhangxyz merged 2 commits into
mainfrom
copilot/remove-mypy-and-update-files
Dec 20, 2025
Merged

Remove unused mypy dependency#124
hzhangxyz merged 2 commits into
mainfrom
copilot/remove-mypy-and-update-files

Conversation

Copilot AI commented Dec 20, 2025

Copy link
Copy Markdown
Contributor

Mypy was configured as a dev dependency but not used in pre-commit hooks, CI workflows, or any build scripts.

Changes:

  • Removed mypy~=1.18.2 from pyproject.toml dev dependencies
  • Removed mypy cache patterns (.mypy_cache/, .dmypy.json, dmypy.json) from .gitignore
  • Regenerated uv.lock to remove mypy and mypy-extensions packages (56 lines removed)

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • astral.sh
    • Triggering command: /usr/bin/curl curl -LsSf REDACTED (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

Original prompt

This section details on the original issue you should resolve

<issue_title>删除mypy,目前并没有使用.</issue_title>
<issue_description>- .gitignore

  • pyproject.toml
  • uv.lock

这几个需要更新。</issue_description>

Comments on the Issue (you are @copilot in this section)


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Co-authored-by: hzhangxyz <11623447+hzhangxyz@users.noreply.github.com>
Copilot AI changed the title [WIP] Remove mypy and update related files Remove unused mypy dependency Dec 20, 2025
Copilot AI requested a review from hzhangxyz December 20, 2025 03:45
@hzhangxyz
hzhangxyz marked this pull request as ready for review December 20, 2025 03:46
Copilot AI review requested due to automatic review settings December 20, 2025 03:46

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 removes the unused mypy type checker dependency from the project. Mypy was configured as a development dependency but was not actually being utilized in pre-commit hooks, CI workflows, or build scripts.

Key changes:

  • Removed mypy from development dependencies in pyproject.toml
  • Cleaned up mypy-related ignore patterns from .gitignore
  • Updated uv.lock to reflect the removal of mypy and its transitive dependency mypy-extensions

Reviewed changes

Copilot reviewed 1 out of 3 changed files in this pull request and generated no comments.

File Description
pyproject.toml Removed mypy~=1.18.2 from the dev optional dependencies list
.gitignore Removed mypy cache patterns (.mypy_cache/, .dmypy.json, dmypy.json) and updated the Python tools comment to reflect mypy's removal
uv.lock Removed 56 lines containing the mypy and mypy-extensions package definitions and their associated wheel distributions

The changes are clean, consistent, and properly synchronized across all three files. No issues were identified during the review.


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

@hzhangxyz
hzhangxyz merged commit 16e522d into main Dec 20, 2025
79 checks passed
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.

删除mypy,目前并没有使用.

3 participants