Skip to content

Type fix base#419

Merged
Erotemic merged 22 commits intopyutils:mainfrom
Erotemic:type-fix-base
Feb 8, 2026
Merged

Type fix base#419
Erotemic merged 22 commits intopyutils:mainfrom
Erotemic:type-fix-base

Conversation

@Erotemic
Copy link
Member

@Erotemic Erotemic commented Feb 8, 2026

This update has a pretty large diff, but there should be no functional changes. The main goal is to improve the type annotations in the library and remove the extra pyi files.

The type annotations had lots of AI assistance, but there was a lot of manual oversight and edits.

Its likely that these can be cleaned up, but I will likely do that in a different PR after I get something stable that type checks. After this I plan to run ruff format over the code, which will produce another huge diff, but hopefully that will facilitate easier maintenance moving forward.

Test User added 11 commits February 8, 2026 18:31
- Set PIP_NO_BINARY=Cython to build Cython from source, avoiding mmap issues
- Fall back to non-editable install if editable install fails due to .so mapping errors
- These changes fix compatibility with container environments that have memory mapping restrictions

This allows the tests to run properly and fail on actual test logic rather than environment issues.
@codecov
Copy link

codecov bot commented Feb 8, 2026

Codecov Report

❌ Patch coverage is 88.88889% with 39 lines in your changes missing coverage. Please review.
✅ Project coverage is 89.06%. Comparing base (60e928f) to head (8e90623).
⚠️ Report is 30 commits behind head on main.

Files with missing lines Patch % Lines
line_profiler/autoprofile/util_static.py 40.90% 26 Missing ⚠️
line_profiler/autoprofile/profmod_extractor.py 78.94% 2 Missing and 2 partials ⚠️
line_profiler/toml_config.py 90.32% 2 Missing and 1 partial ⚠️
line_profiler/cli_utils.py 92.59% 1 Missing and 1 partial ⚠️
line_profiler/profiler_mixin.py 94.44% 1 Missing and 1 partial ⚠️
...ne_profiler/autoprofile/ast_profile_transformer.py 94.44% 0 Missing and 1 partial ⚠️
line_profiler/autoprofile/autoprofile.py 91.66% 0 Missing and 1 partial ⚠️
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #419      +/-   ##
==========================================
+ Coverage   87.56%   89.06%   +1.49%     
==========================================
  Files          18       20       +2     
  Lines        1641     2258     +617     
  Branches      348      474     +126     
==========================================
+ Hits         1437     2011     +574     
- Misses        149      184      +35     
- Partials       55       63       +8     
Files with missing lines Coverage Δ
line_profiler/__main__.py 100.00% <100.00%> (ø)
line_profiler/_diagnostics.py 100.00% <100.00%> (ø)
line_profiler/autoprofile/ast_tree_profiler.py 100.00% <100.00%> (ø)
line_profiler/autoprofile/eager_preimports.py 93.95% <100.00%> (+0.81%) ⬆️
line_profiler/autoprofile/line_profiler_utils.py 88.23% <100.00%> (+3.61%) ⬆️
line_profiler/autoprofile/run_module.py 100.00% <100.00%> (ø)
line_profiler/explicit_profiler.py 80.12% <100.00%> (-13.78%) ⬇️
line_profiler/ipython_extension.py 98.19% <100.00%> (+38.76%) ⬆️
line_profiler/line_profiler.py 98.94% <100.00%> (-0.72%) ⬇️
line_profiler/line_profiler_utils.py 88.88% <100.00%> (+1.38%) ⬆️
... and 8 more

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 88590a0...8e90623. Read the comment docs.

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

@Erotemic Erotemic merged commit 64c0841 into pyutils:main Feb 8, 2026
45 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.

1 participant