Skip to content

Conversation

@eumiro
Copy link

@eumiro eumiro commented Mar 13, 2025

Replace {time.strftime('%X')} with {time:%X} in the asyncio-task docs to lower the complexity of the code that is not the main topic of the docs and not stealing the attention of the reader.


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

Copy link
Contributor

@hauntsaninja hauntsaninja left a comment

Choose a reason for hiding this comment

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

Thanks. I like the intention, but the time symbol is introduced with import time — we should make sure that a user could fairly easily run the code and produce the expected output themselves.

@eumiro
Copy link
Author

eumiro commented Mar 14, 2025

You're right, @hauntsaninja . This really stole my attention.

@eumiro eumiro closed this Mar 14, 2025
@eumiro eumiro deleted the asyncio-docs-strftime branch March 14, 2025 17:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting review docs Documentation in the Doc dir skip news

Projects

Status: Todo

Development

Successfully merging this pull request may close these issues.

2 participants