Skip to content

Conversation

@larsevj
Copy link
Collaborator

@larsevj larsevj commented Nov 21, 2025

Will wait for new res2df release.

@codecov-commenter
Copy link

codecov-commenter commented Nov 21, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 83.46%. Comparing base (1a0dd3f) to head (edade29).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #838   +/-   ##
=======================================
  Coverage   83.46%   83.46%           
=======================================
  Files          49       49           
  Lines        7293     7293           
=======================================
  Hits         6087     6087           
  Misses       1206     1206           

☔ 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.

@larsevj larsevj force-pushed the remove_res2df_ignore branch from da02b68 to edade29 Compare December 11, 2025 13:38
@larsevj larsevj marked this pull request as ready for review December 11, 2025 13:56
@larsevj larsevj requested review from a team, alifbe and Copilot December 11, 2025 13:56
Copy link

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

This PR removes res2df.* from the mypy ignore list, enabling type checking for the res2df library. This change indicates that res2df now provides type hints or type stubs, allowing mypy to perform type checking on code that uses this library.

Key Changes

  • Enables mypy type checking for res2df by removing it from the ignore_missing_imports list in the mypy configuration

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

Copy link
Collaborator

@alifbe alifbe left a comment

Choose a reason for hiding this comment

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

Cool 👍

@larsevj larsevj merged commit 31f65ed into main Dec 11, 2025
12 checks passed
@larsevj larsevj deleted the remove_res2df_ignore branch December 11, 2025 14:00
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.

4 participants