Skip to content

gh-148352: Add more colour to calendar CLI output#148354

Open
hugovk wants to merge 6 commits intopython:mainfrom
hugovk:3.15-calendar-colour
Open

gh-148352: Add more colour to calendar CLI output#148354
hugovk wants to merge 6 commits intopython:mainfrom
hugovk:3.15-calendar-colour

Conversation

@hugovk
Copy link
Copy Markdown
Member

@hugovk hugovk commented Apr 10, 2026

In Python 3.14, we added an amazing colour highlight for today: #128318.

Let's add a touch more colour.

Before After
Image Image

📚 Documentation preview 📚: https://cpython-previews--148354.org.readthedocs.build/

@hugovk hugovk requested a review from AA-Turner as a code owner April 10, 2026 17:27
@hugovk hugovk added the stdlib Standard Library Python modules in the Lib/ directory label Apr 10, 2026
Co-authored-by: Rihaan Meher <sharktidedev@gmail.com>
parser.add_argument(
"-f", "--first-weekday",
type=int, default=0,
help="weekday (0 is Monday, 6 is Sunday) to start each week (default 0)"
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.

Why stop telling the user the defaults? I think it adds clarity in certain situations :) up to you however

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

On the contrary! 1fe0d93 formats them with ArgumentDefaultsHelpFormatter to add colour :)

image image

@sharktide

This comment was marked as resolved.

Co-authored-by: Rihaan Meher <sharktidedev@gmail.com>
Copy link
Copy Markdown
Contributor

@sharktide sharktide left a comment

Choose a reason for hiding this comment

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

LGTM, thanks @hugovk :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting core review stdlib Standard Library Python modules in the Lib/ directory

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants