Skip to content

Use f-strings in csv docs example #135245

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

Merged
merged 1 commit into from
Jun 8, 2025

Conversation

ezio-melotti
Copy link
Member

@ezio-melotti ezio-melotti commented Jun 8, 2025

Just a minor tweak to the csv module docs to use f-strings instead of .format().


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

@ezio-melotti ezio-melotti self-assigned this Jun 8, 2025
@ezio-melotti ezio-melotti added docs Documentation in the Doc dir skip issue skip news 3.13 bugs and security fixes 3.14 bugs and security fixes 3.15 new features, bugs and security fixes labels Jun 8, 2025
@github-project-automation github-project-automation bot moved this to Todo in Docs PRs Jun 8, 2025
@hauntsaninja hauntsaninja merged commit 2677dd0 into python:main Jun 8, 2025
39 checks passed
@github-project-automation github-project-automation bot moved this from Todo to Done in Docs PRs Jun 8, 2025
@ezio-melotti
Copy link
Member Author

Should this be backported to 3.14 and 3.13 too?

@hauntsaninja
Copy link
Contributor

We can if you want to? Biggest reason to do so would be if you have more changes planned and would like to avoid potential merge conflicts when backporting future PRs

@ezio-melotti ezio-melotti added needs backport to 3.13 bugs and security fixes needs backport to 3.14 bugs and security fixes labels Jun 9, 2025
@miss-islington-app
Copy link

Thanks @ezio-melotti for the PR, and @hauntsaninja for merging it 🌮🎉.. I'm working now to backport this PR to: 3.14.
🐍🍒⛏🤖 I'm not a witch! I'm not a witch!

@miss-islington-app
Copy link

Thanks @ezio-melotti for the PR, and @hauntsaninja for merging it 🌮🎉.. I'm working now to backport this PR to: 3.13.
🐍🍒⛏🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Jun 9, 2025
(cherry picked from commit 2677dd0)

Co-authored-by: Ezio Melotti <ezio.melotti@gmail.com>
@bedevere-app
Copy link

bedevere-app bot commented Jun 9, 2025

GH-135285 is a backport of this pull request to the 3.14 branch.

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Jun 9, 2025
(cherry picked from commit 2677dd0)

Co-authored-by: Ezio Melotti <ezio.melotti@gmail.com>
@bedevere-app bedevere-app bot removed the needs backport to 3.14 bugs and security fixes label Jun 9, 2025
@bedevere-app
Copy link

bedevere-app bot commented Jun 9, 2025

GH-135286 is a backport of this pull request to the 3.13 branch.

@bedevere-app bedevere-app bot removed the needs backport to 3.13 bugs and security fixes label Jun 9, 2025
@ezio-melotti ezio-melotti deleted the csv-docs-f-strings branch June 9, 2025 06:32
hauntsaninja pushed a commit that referenced this pull request Jun 9, 2025
Use f-strings in csv docs example (GH-135245)
(cherry picked from commit 2677dd0)

Co-authored-by: Ezio Melotti <ezio.melotti@gmail.com>
hauntsaninja pushed a commit that referenced this pull request Jun 9, 2025
Use f-strings in csv docs example (GH-135245)
(cherry picked from commit 2677dd0)

Co-authored-by: Ezio Melotti <ezio.melotti@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3.13 bugs and security fixes 3.14 bugs and security fixes 3.15 new features, bugs and security fixes docs Documentation in the Doc dir skip issue skip news
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants