build(deps-dev): bump mypy to 1.16.0 #1480

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Merged

Lee-W merged 2 commits into commitizen-tools:refactors from gbaian10:upgrade-mypy

May 31, 2025

Merged

build(deps-dev): bump mypy to 1.16.0 #1480

Lee-W merged 2 commits into commitizen-tools:refactors from gbaian10:upgrade-mypy

May 31, 2025

Conversation

gbaian10

Copy link

Contributor

@gbaian10 gbaian10 commented May 31, 2025

Description

Resolves #1476

Upgrade mypy to 1.16 #1476

Checklist

I have read the contributing guidelines

Code Changes

Add test cases to all the changes you introduce
Run poetry all locally to ensure this change passes linter check and tests
Manually test the changes:
- Verify the feature/bug fix works as expected in real-world scenarios
- Test edge cases and error conditions
- Ensure backward compatibility is maintained
- Document any manual testing steps performed
Update the documentation for the changes

Documentation Changes

Run poetry doc locally to ensure the documentation pages renders correctly
- Check if there are any broken links in the documentation

When running poetry doc, any broken internal documentation links will be reported in the console output like this:
INFO - Doc file 'config.md' contains a link 'commands/bump.md#-post_bump_hooks', but the doc 'commands/bump.md' does not contain an anchor '#-post_bump_hooks'.

Expected Behavior

Steps to Test This Pull Request

Additional Context

@gbaian10 gbaian10 requested review from woile, Lee-W and noirbizarre as code owners

May 31, 2025 04:37

@github-actions github-actions bot added the pr-status: wait-for-review label

May 31, 2025

Lee-W

Lee-W approved these changes

May 31, 2025

View reviewed changes

@Lee-W Lee-W added pr-status: ready-to-merge and removed pr-status: wait-for-review labels

May 31, 2025

@Lee-W

Copy link

Member

Lee-W commented May 31, 2025

we'll need to fix the style check failure due to this upgrade before merging it

@Lee-W Lee-W added pr-status: wait-for-modification and removed pr-status: ready-to-merge labels

May 31, 2025

@Lee-W Lee-W self-requested a review

May 31, 2025 04:39

@github-actions github-actions bot added the pr-status: wait-for-review label

May 31, 2025

@codecov Codecov

Copy link

codecov bot commented May 31, 2025 •

edited

Loading

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 97.82%. Comparing base (c0220cd) to head (952c699).
Report is 2 commits behind head on refactors.

Additional details and impacted files

@@ Coverage Diff @@
## refactors #1480 +/- ##
==========================================
 Coverage 97.82% 97.82% 
==========================================
 Files 57 57 
 Lines 2618 2618 
==========================================
 Hits 2561 2561 
 Misses 57 57