◐ Shell
reader mode source ↗
Skip to content

gh-88037: Move port validation logic to parsing time#25774

Open
miguendes wants to merge 1 commit into
python:mainfrom
miguendes:bpo-43871
Open

gh-88037: Move port validation logic to parsing time#25774
miguendes wants to merge 1 commit into
python:mainfrom
miguendes:bpo-43871

Conversation

@miguendes

@miguendes miguendes commented May 1, 2021 •
edited by bedevere-bot
Loading

Copy link
Copy Markdown
Contributor

@miguendes miguendes force-pushed the bpo-43871 branch 4 times, most recently from 7cf9ace to cf3913e Compare May 1, 2021 12:54
@github-actions

github-actions Bot commented Jun 3, 2021

Copy link
Copy Markdown

This PR is stale because it has been open for 30 days with no activity.

@github-actions github-actions Bot added the stale Stale PR or inactive for long period of time. label Jun 3, 2021
@miguendes miguendes force-pushed the bpo-43871 branch 2 times, most recently from 5ebd099 to 3419baa Compare July 10, 2021 09:03
MaxwellDupre
MaxwellDupre approved these changes Jul 13, 2022
View reviewed changes

@MaxwellDupre MaxwellDupre left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Hide comment

Nice addition

@github-actions github-actions Bot removed the stale Stale PR or inactive for long period of time. label Aug 7, 2022
@kenballus kenballus mentioned this pull request Aug 16, 2022
Closed
@gpshead gpshead self-requested a review September 13, 2022 22:12
@kenballus kenballus mentioned this pull request Oct 17, 2022
Merged
@kenballus

kenballus commented Jan 26, 2023

Copy link
Copy Markdown
Contributor

Any update on this? I need this behavior, and have implemented a fix myself. Should I submit another PR?

@gpshead gpshead changed the title bpo-43871: Move port validation logic to parsing time Jan 26, 2023
@gpshead gpshead self-assigned this Jan 26, 2023
@gpshead gpshead added type-bug stdlib Standard Library Python modules in the Lib/ directory labels Jan 26, 2023
@gpshead

gpshead commented May 20, 2023

Copy link
Copy Markdown
Member

Any update on this? I need this behavior, and have implemented a fix myself. Should I submit another PR?

@kenballus indeed, looks like this one has stalled. a new up to date PR against main (based on this if that makes sense to do) would help.

@p-alik p-alik mannequin mentioned this pull request Nov 23, 2023
Open
@github-actions

github-actions Bot commented Apr 9, 2026

Copy link
Copy Markdown

This PR is stale because it has been open for 30 days with no activity.

@github-actions github-actions Bot added the stale Stale PR or inactive for long period of time. label Apr 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@akulakov akulakov akulakov left review comments

@gpshead gpshead Awaiting requested review from gpshead

+1 more reviewer

@MaxwellDupre MaxwellDupre MaxwellDupre approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@gpshead gpshead

Labels

awaiting core review stale Stale PR or inactive for long period of time. stdlib Standard Library Python modules in the Lib/ directory type-bug An unexpected behavior, bug, or error

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

8 participants

@miguendes @kenballus @gpshead @akulakov @MaxwellDupre @the-knights-who-say-ni @ezio-melotti @bedevere-bot