{{ message }}
gh-140212: Add html for year-month option in Calendar#140230
Merged
Conversation
|
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 |
Contributor
Author
|
Note that the html produced in I didn't do this at this stage because it would actually be a larger change, and I'm not sure this will be merged regardless. Let me know if I should extract the common code out. |
StanFromIreland
left a comment
StanFromIreland
left a comment
Member
There was a problem hiding this comment.
Please add a blurb and docs.
Contributor
Author
|
@StanFromIreland I have added a blurb and updated documentation for Calendar's argparser. I was not sure if the blurb was supposed to go in I have updated the docs as well. I think it's best to squash the commits before a hypothetical merging. I'll do so upon request. |
Member
|
It should be in Library. Squash is done by the commiter, please don't force push, it only causes confusion. |
Contributor
Author
|
@StanFromIreland I have moved the blurb to Library. |
pgdr
changed the title
hugovk
reviewed
Oct 22, 2025
Contributed by hugovk Co-authored-by: Hugo van Kemenade <1324225+hugovk@users.noreply.github.com>
Member
|
Please could you also list this in What's New? |
Contributor
Author
|
@hugovk It's done (given that the entry is acceptable). |
Contributor
Author
|
@AA-Turner Let me know if there is anything I can do at this point to get this PR merged. |
Member
|
Let's merge, thanks! |
|
Member
StanFromIreland
pushed a commit
to StanFromIreland/cpython
that referenced
this pull request
Dec 6, 2025
…40230) Co-authored-by: Hugo van Kemenade <1324225+hugovk@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Running
python3.13 -m calendar 2025pretty-prints the calendar for all of 2025. If we add the month at the end, it only prints the month.python3.13 -m calendar 2025It is possible to get HTML output by using
python -m calendar -t html 2025, which gives the calender for 2025 in HTML format.However, if we add the month,
python -m calendar -t html 2025 10it fails:This PR simply adds support for HTML-formatting the year-month option.
A test is included, and one existing test is removed: a test verifying that the year month option fails.