bpo-44249 Update README.rst #26385
bpo-44249 Update README.rst #26385
Conversation
This is too trivial for a news entry IMHO. I also, forget if we can really merge changes if authors have not signed the CLA. |
I signed CLA , look over it again |
workload. This is necessary in order to profile the interpreter execution. | ||
Note also that any output, both stdout and stderr, that may appear at this step | ||
workload. This is necessary to profile the interpreter's execution. | ||
Note also that any output, both stdout, and stderr, that may appear at this step |
zware
May 27, 2021
Member
This should not change; it's not a list, it's just a sub-section of the sentence. I'm not sure this note is true anymore, though.
is suppressed. | ||
|
||
The final step is to build the actual interpreter, using the information | ||
collected from the instrumented one. The end result will be a Python binary | ||
collected from the instrumented one. The result will be a Python binary |
@@ -0,0 +1 @@ | |||
updated and corrected readme with some grammatical mistakes |
zware
May 27, 2021
Member
This entry is definitely not needed, but NEWS entries should also have proper grammar :)
A Python core developer has requested some changes be made to your pull request before we can consider merging it. If you could please address their requests along with any other requests in other reviews from core developers that would be appreciated. Once you have made the requested changes, please leave a comment on this pull request containing the phrase |
@zware, please do the honors of closing the PR unless @Ayushparikh-code has any rebuttals for the changes proposed. Edit: I think I misread some of the reviews I guess to mean the change wasn't necessary, my bad. |
Co-authored-by: Zachary Ware <zachary.ware@gmail.com>
Done |
I have made the requested changes; please review again |
Thanks for making the requested changes! @zware: please review the changes made to this pull request. |
(●'◡'●) |
flags for each flavor. Note that this is just an intermediary step. The | ||
binary resulting from this step is not good for real-life workloads as it has |
terryjreedy
May 28, 2021
•
Member
These two changes and one other are correct. Will merge when CI done.
workload. This is necessary in order to profile the interpreter execution. | ||
Note also that any output, both stdout and stderr, that may appear at this step | ||
workload. This is necessary to profile the interpreter's execution. | ||
Note also that any output, both stdout, and stderr, that may appear at this step |
@nanjekyejoannah Trivial changes like this, with no creative content, do not need a CLA, But I still think getting it is better. If nothing else, so the matter is taken care of for next suggestion. |
Thanks @Ayushparikh-code for the PR, and @terryjreedy for merging it |
GH-26434 is a backport of this pull request to the 3.10 branch. |
(cherry picked from commit acac6c7) Co-authored-by: Ayush Parikh <ayushparikh332@gmail.com>
(cherry picked from commit acac6c7) Co-authored-by: Ayush Parikh <ayushparikh332@gmail.com>
GH-26435 is a backport of this pull request to the 3.9 branch. |
https://bugs.python.org/issue44249