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>
@bedevere-bot
Copy link

⚠️⚠️⚠️ Buildbot failure ⚠️⚠️⚠️

Hi! The buildbot aarch64 Fedora Stable Refleaks 3.13 (tier-2) has failed when building commit 8c28241.

What do you need to do:

  1. Don't panic.
  2. Check the buildbot page in the devguide if you don't know what the buildbots are or how they work.
  3. Go to the page of the buildbot that failed (https://buildbot.python.org/#/builders/1512/builds/691) and take a look at the build logs.
  4. Check if the failure is related to this commit (8c28241) or if it is a false positive.
  5. If the failure is related to this commit, please, reflect that on the issue and make a new Pull Request with a fix.

You can take a look at the buildbot page here:

https://buildbot.python.org/#/builders/1512/builds/691

Failed tests:

  • test_perf_profiler

Failed subtests:

  • test_python_calls_appear_in_the_stack_if_perf_activated - test.test_perf_profiler.TestPerfProfilerWithDwarf.test_python_calls_appear_in_the_stack_if_perf_activated

Summary of the results of the build (if available):

==

Click to see traceback logs
Traceback (most recent call last):
  File "/home/buildbot/buildarea/3.13.cstratak-fedora-stable-aarch64.refleak/build/Lib/test/test_perf_profiler.py", line 367, in test_python_calls_appear_in_the_stack_if_perf_activated
    self.assertEqual(stderr, "")
    ~~~~~~~~~~~~~~~~^^^^^^^^^^^^
AssertionError: 'Warning:\nProcessed 675 events and lost 1[34 chars]\n\n' != ''
- Warning:
- Processed 675 events and lost 1 chunks!
- 
- Check IO/CPU overload!
- 


Traceback (most recent call last):
  File "/home/buildbot/buildarea/3.13.cstratak-fedora-stable-aarch64.refleak/build/Lib/test/test_perf_profiler.py", line 367, in test_python_calls_appear_in_the_stack_if_perf_activated
    self.assertEqual(stderr, "")
    ~~~~~~~~~~~~~~~~^^^^^^^^^^^^
AssertionError: 'Warning:\nProcessed 685 events and lost 1[34 chars]\n\n' != ''
- Warning:
- Processed 685 events and lost 1 chunks!
- 
- Check IO/CPU overload!
- 

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.

4 participants
pFad - Phonifier reborn

Pfad - The Proxy pFad of © 2024 Garber Painting. All rights reserved.

Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.


Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy