Skip to content
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

Remove limit in Calendar CLI help message for year arg #114719

Merged
merged 1 commit into from
Jan 29, 2024
Merged

Remove limit in Calendar CLI help message for year arg #114719

merged 1 commit into from
Jan 29, 2024

Conversation

planet36
Copy link
Contributor

The limit was removed in 66c88ce.

It was requested here #112241 (comment) that a trivial PR be made for this change.

@bedevere-app
Copy link

bedevere-app bot commented Jan 29, 2024

Most changes to Python require a NEWS entry. Add one using the blurb_it web app or the blurb command-line tool.

If this change has little impact on Python users, wait for a maintainer to apply the skip news label instead.

Copy link
Member

@serhiy-storchaka serhiy-storchaka left a comment

Choose a reason for hiding this comment

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

LGTM.

@serhiy-storchaka serhiy-storchaka merged commit 0f54ee4 into python:main Jan 29, 2024
34 checks passed
@miss-islington-app
Copy link

Thanks @planet36 for the PR, and @serhiy-storchaka for merging it 🌮🎉.. I'm working now to backport this PR to: 3.11, 3.12.
🐍🍒⛏🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Jan 29, 2024
The limit was removed in 66c88ce (pythonGH-4109).
(cherry picked from commit 0f54ee4)

Co-authored-by: Steven Ward <[email protected]>
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Jan 29, 2024
The limit was removed in 66c88ce (pythonGH-4109).
(cherry picked from commit 0f54ee4)

Co-authored-by: Steven Ward <[email protected]>
@bedevere-app
Copy link

bedevere-app bot commented Jan 29, 2024

GH-114722 is a backport of this pull request to the 3.12 branch.

@bedevere-app bedevere-app bot removed the needs backport to 3.12 bug and security fixes label Jan 29, 2024
@bedevere-app
Copy link

bedevere-app bot commented Jan 29, 2024

GH-114723 is a backport of this pull request to the 3.11 branch.

@bedevere-app bedevere-app bot removed the needs backport to 3.11 only security fixes label Jan 29, 2024
serhiy-storchaka pushed a commit that referenced this pull request Jan 29, 2024
) (GH-114722)

The limit was removed in 66c88ce (GH-4109).
(cherry picked from commit 0f54ee4)

Co-authored-by: Steven Ward <[email protected]>
serhiy-storchaka pushed a commit that referenced this pull request Jan 29, 2024
) (GH-114723)

The limit was removed in 66c88ce (GH-4109).
(cherry picked from commit 0f54ee4)

Co-authored-by: Steven Ward <[email protected]>
@planet36 planet36 deleted the cal-cli-help-year-limit branch January 30, 2024 00:55
aisk pushed a commit to aisk/cpython that referenced this pull request Feb 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants