Skip to content

Bump Mypy#2181

Draft
mattwthompson wants to merge 4 commits into
mainfrom
bump-mypy
Draft

Bump Mypy#2181
mattwthompson wants to merge 4 commits into
mainfrom
bump-mypy

Conversation

@mattwthompson
Copy link
Copy Markdown
Member

@codecov
Copy link
Copy Markdown

codecov Bot commented May 14, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 94.19%. Comparing base (6e2fc35) to head (b17860b).

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@mattwthompson mattwthompson requested a review from Copilot May 14, 2026 19:04
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Note

Copilot was unable to run its full agentic suite in this review.

Updates type-checking configuration to support a newer Mypy version.

Changes:

  • Remove NumPy Mypy plugin configuration from pyproject.toml
  • Add a Mypy ignore for a NetworkX node iteration typing issue
  • Switch OpenEye dev env to install Mypy via pip (and update its pinned version)

Reviewed changes

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

File Description
pyproject.toml Adjusts Mypy configuration (removes NumPy plugin entry).
openff/toolkit/topology/molecule.py Adds a Mypy suppression on a NetworkX nodes iteration.
devtools/conda-envs/openeye.yaml Changes how Mypy is installed/pinned in the OpenEye conda env.

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

Comment thread openff/toolkit/topology/molecule.py
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.

2 participants