-
-
Notifications
You must be signed in to change notification settings - Fork 32.2k
Fix spacing in tarfile docs #20601
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
Fix spacing in tarfile docs #20601
Conversation
GH- [bpo-39128](https://bugs.python.org/issue39128): happy eyeballs description GH- [3.9] 39128 - happy eyeballs description (GH-18624) GH- [3.8] 39128 - happy eyeballs description (GH-18624) https://bugs.python.org/issue39128 (cherry picked from commit 8af4712) Co-authored-by: idomic <[email protected]>
Trying to decode an invalid string with the punycode codec shoud raise UnicodeError. (cherry picked from commit ba22e8f) Co-authored-by: Berker Peksag <[email protected]>
Python 3.8.2
…GH-18657) (cherry picked from commit d644891) Co-authored-by: Steve Dower <[email protected]>
(cherry picked from commit 57c7a0b) Co-authored-by: Inada Naoki <[email protected]>
Co-authored-by: Ammar Askar <[email protected]>
Previously, the button-up part of selecting with a mouse was treated as a click that meant 'jump' to this line, which modified the context and undid the selection (cherry picked from commit c705fd1) Co-authored-by: Terry Jan Reedy <[email protected]>
(cherry picked from commit 916895f) Co-authored-by: Terry Jan Reedy <[email protected]>
…8598) (cherry picked from commit c2f7eb2) Co-authored-by: Shantanu <[email protected]>
Also fixes some potential Nuget build issues. (cherry picked from commit 03153dd) Co-authored-by: Steve Dower <[email protected]>
…H-18338) * bpo-39548: Fix handling of 'WWW-Authenticate' header for Digest authentication - The 'qop' value in the 'WWW-Authenticate' header is optional. The presence of 'qop' in the header should be checked before its value is parsed with 'split'. Signed-off-by: Stephen Balousek <[email protected]> * bpo-39548: Fix handling of 'WWW-Authenticate' header for Digest authentication - Add NEWS item Signed-off-by: Stephen Balousek <[email protected]> * Update Misc/NEWS.d/next/Library/2020-02-06-05-33-52.bpo-39548.DF4FFe.rst Co-Authored-By: Brandt Bucher <[email protected]> Co-authored-by: Brandt Bucher <[email protected]> (cherry picked from commit 5e260e0) Co-authored-by: Stephen Balousek <[email protected]>
Fix compileall.compile_dir() ddir= behavior on sub-packages. Fixes compileall.compile_dir's ddir parameter and compileall command line flag `-d` to no longer write the wrong pathname to the generated pyc file for submodules beneath the root of the directory tree being compiled. This fixes a regression introduced with Python 3.5. Tests backported from GH 0267335, the implementation is different due to intervening code changes. But still quiet simple. Why was the bug ever introduced? The refactoring to add parallel execution kept the ddir -> dfile computations but discarded the results instead of sending them to compile_file(). This fixes that. Lack of tests meant this went unnoticed.
Automerge-Triggered-By: @pablogsal. (cherry picked from commit 217dce9) Co-authored-by: Hakan Çelik <[email protected]>
(cherry picked from commit 1f577ce) Co-authored-by: Hai Shi <[email protected]>
_PyEval_SetAsyncGenFinalizer() and _PyEval_SetAsyncGenFirstiter() didn't include proper error handling for their PySys_Audit() calls. Co-authored-by: Zackery Spytz <[email protected]>
…-17666) Open issue in the BPO indicated a desire to make the implementation of codecs.open() at parity with io.open(), which implements a try/except to assure file stream gets closed before an exception is raised. (cherry picked from commit 2565ede) Co-authored-by: Chris A <[email protected]>
Co-authored-by: blurb-it[bot] <43283697+blurb-it[bot]@users.noreply.github.com> (cherry picked from commit 4482337) Co-authored-by: Lidi Zheng <[email protected]>
Automerge-Triggered-By: @zooba (cherry picked from commit 1382c32) Co-authored-by: Erlend Egeberg Aasland <[email protected]>
…dows (GH-18724) (cherry picked from commit ce3a498) Co-authored-by: Steve Dower <[email protected]>
…H-18761) (cherry picked from commit 2d2f855) Co-authored-by: Serhiy Storchaka <[email protected]>
Automerge-Triggered-By: @pablogsal (cherry picked from commit 6df421f) Co-authored-by: Pablo Galindo <[email protected]>
(cherry picked from commit c610d97) Co-authored-by: Erlend Egeberg Aasland <[email protected]>
Leave selection when right click within. This exception to clearing selections when right-clicking was omitted from the previous commit, 4ca060d. I did not realize that this completely disabled the context menu entries, and I should have merged a minimal fix immediately. An automated test should follow. (cherry picked from commit 97e4e0f) Co-authored-by: Terry Jan Reedy <[email protected]>
…GH-20149) Reference to PySide has been removed has it is for Qt 4, which has reached end of life. (cherry picked from commit 4649202) Co-authored-by: Samuel Gaist <[email protected]>
…H-20483) (cherry picked from commit 735d902) Co-authored-by: Florian Dahlitz <[email protected]>
Fix :mod:`ssl`` code to be compatible with OpenSSL 1.1.x builds that use ``no-deprecated`` and ``--api=1.1.0``. Note: Tests assume full OpenSSL API and fail with limited API. Signed-off-by: Christian Heimes <[email protected]> Co-authored-by: Mark Wright <[email protected]> (cherry picked from commit a871f69) Co-authored-by: Christian Heimes <[email protected]>
(cherry picked from commit c896666) Co-authored-by: Zackery Spytz <[email protected]>
(cherry picked from commit fe5dd78) Co-authored-by: Steve Dower <[email protected]>
Before ``` content.txt is 42 bytes in size and isa regular file. folder is 420 bytes in size and isa directory. magic is 4200 bytes in size and issomething else. ``` After: ``` content.txt is 42 bytes in size and is a regular file. folder is 420 bytes in size and is a directory. magic is 4200 bytes in size and is something else. ```
Hello, and thanks for your contribution! I'm a bot set up to make sure that the project can legally accept this contribution by verifying everyone involved has signed the PSF contributor agreement (CLA). Recognized GitHub usernameWe couldn't find a bugs.python.org (b.p.o) account corresponding to the following GitHub usernames: This might be simply due to a missing "GitHub Name" entry in one's b.p.o account settings. This is necessary for legal reasons before we can look at this contribution. Please follow the steps outlined in the CPython devguide to rectify this issue. You can check yourself to see if the CLA has been received. Thanks again for the contribution, we look forward to reviewing it! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @HarshaLaxman, thanks for contributing to Python!
Like @the-knights-who-say-ni said, you will need to sign the CLA so that this contribution can be accepted in Python. You can follow the steps given in the Python Dev Guide.
You will also need to change the target branch of your Pull Request, all PR are made against master
and your fix will be backported to the maintenance branches once it's accepted.
@@ -784,7 +784,7 @@ How to read a gzip compressed tar archive and display some member information:: | |||
import tarfile | |||
tar = tarfile.open("sample.tar.gz", "r:gz") | |||
for tarinfo in tar: | |||
print(tarinfo.name, "is", tarinfo.size, "bytes in size and is", end="") | |||
print(tarinfo.name, "is", tarinfo.size, "bytes in size and is ", end="") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe this statement could use an f-string?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ack. Still this functional improvement is valuable.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
@HarshaLaxman , please sign the CLA |
I changed the target branch to Master as requested. I did sign the CLA yesterday with bugs.python.org username |
Please make sure your GitHub username is listed in the “Your Details” section at b.p.o: https://cloud.githubusercontent.com/assets/2680980/23276970/d14a380c-f9d1-11e6-883d-e13b6b211239.png Also, thanks for changing the target branch but there should be only one commit in the PR, it took all changes that were in the branch. |
My GitHub username is in the "Your Details" section, but even b.p.o doesn't recognize that I signed the CLA. Comparing to the screenshot you shared, my page does not include "Contributor Form Received", although I got email confirmation and ewa was cc'd. For changing the target branch, I can create a new PR against master once I figure out this CLA stuff? |
Something is wrong with this PR. I suggest, pushing (force pushing) a clean version, or dropping this to create a new PR. Only one PR against |
Looks like my CLA is acknowledged now. This can be closed, thanks for the assistance :) |
Before
After: