Skip to content

gh-99815: remove unused 'invalid' sentinel value and code that checks for it in inspect.signature parsing - #21104

Merged
iritkatriel merged 2 commits into
python:mainfrom
asottile:inspect_signature_simplify
Nov 27, 2022
Merged

gh-99815: remove unused 'invalid' sentinel value and code that checks for it in inspect.signature parsing#21104
iritkatriel merged 2 commits into
python:mainfrom
asottile:inspect_signature_simplify

Conversation

@asottile

@asottileasottile commented Jun 24, 2020•
edited by bedevere-bot
Loading

Copy link
Copy Markdown
Contributor

@asottile
asottileforce-pushed the inspect_signature_simplify branch from 5309efa to 847fd9fCompareSeptember 3, 2020 18:10
@asottile
asottileforce-pushed the inspect_signature_simplify branch from 847fd9f to e569169CompareNovember 2, 2021 02:24
@asottile
asottileforce-pushed the inspect_signature_simplify branch from e569169 to 477c2ebCompareJuly 8, 2022 16:27
@asottile
asottileforce-pushed the inspect_signature_simplify branch from 477c2eb to 83f9192CompareAugust 5, 2022 20:22
@asottile

asottile commented Aug 29, 2022

Copy link
Copy Markdown
ContributorAuthor

@1st1 (inspect expert) -- been sitting on this small fix for a few years -- any chance of a review ?

@asottile
asottileforce-pushed the inspect_signature_simplify branch from 83f9192 to e6eaf8eCompareOctober 5, 2022 21:16
@iritkatrieliritkatriel changed the title Simplify routine in inspect.signature parsingremove unused 'invalid' variable in inspect.signature parsingNov 27, 2022
@iritkatrieliritkatriel changed the title remove unused 'invalid' variable in inspect.signature parsingremove unused 'invalid' value in inspect.signature parsingNov 27, 2022
@iritkatrieliritkatriel changed the title remove unused 'invalid' value in inspect.signature parsinggh-99815: remove unused 'invalid' sentinel value and code that checks for it in inspect.signature parsingNov 27, 2022
@bedevere-botbedevere-bot mentioned this pull request Nov 27, 2022
Closed
@iritkatriel
iritkatriel merged commit ac115b5 into python:mainNov 27, 2022
@JelleZijlstraJelleZijlstra mentioned this pull request Dec 12, 2022
Merged
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@iritkatrieliritkatrieliritkatriel approved these changes
+1 more reviewer
@eamanueamanueamanu approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

skip news

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

7 participants

@asottile@iritkatriel@eamanu@the-knights-who-say-ni@ezio-melotti@bedevere-bot@kumaraditya303