Skip to content

Add typing to _util.py#2726

Closed
CoolCat467 wants to merge 1 commit intopython-trio:masterfrom
CoolCat467:typing-util
Closed

Add typing to _util.py#2726
CoolCat467 wants to merge 1 commit intopython-trio:masterfrom
CoolCat467:typing-util

Conversation

@CoolCat467
Copy link
Member

This PR adds type annotations to _util.py

I added a bunch of Any annotations for generic_function, I can't think of anything better at the moment.

I would love feedback and any comments on how this could be improved to be more accurate.

@codecov
Copy link

codecov bot commented Jul 30, 2023

Codecov Report

Merging #2726 (abdc65e) into master (cf1f3c7) will decrease coverage by 0.02%.
The diff coverage is 90.00%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2726      +/-   ##
==========================================
- Coverage   98.89%   98.88%   -0.02%     
==========================================
  Files         113      113              
  Lines       16600    16605       +5     
  Branches     3014     3015       +1     
==========================================
+ Hits        16417    16420       +3     
- Misses        126      127       +1     
- Partials       57       58       +1     
Files Changed Coverage Δ
trio/_util.py 98.34% <90.00%> (-1.66%) ⬇️

@A5rocks
Copy link
Contributor

A5rocks commented Jul 30, 2023

Oh geh, another overlap with #2719!

Mind giving that a look-over? Given you've done the same process yourself, you should probably have opinions on how it should be!

@A5rocks A5rocks added the typing Adding static types to trio's interface label Jul 30, 2023
@jakkdl
Copy link
Member

jakkdl commented Jul 30, 2023

Closing this one to focus on #2719

@jakkdl jakkdl closed this Jul 30, 2023
@CoolCat467 CoolCat467 deleted the typing-util branch January 9, 2024 18:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

typing Adding static types to trio's interface

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants