Change by Jay Crotts :
--
resolution: -> rejected
stage: patch review -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.or
Jay Crotts added the comment:
No worries, I think all of your points make sense, especially number one after
looking at the patch again. I looked at the docs again, and there is even an
example of another built in being patched.
Thanks for taking the time to review it.
I'm okay
Change by Jay Crotts :
--
pull_requests: +12700
___
Python tracker
<https://bugs.python.org/issue33722>
___
___
Python-bugs-list mailing list
Unsubscribe:
Jay Crotts added the comment:
Yeah the PR had been stale for a while, and I re-based my fork without closing
the PR, so when I pushed it to the fork it updated the PR reviewer list. Silly
mistake by me, I should have made sure the diff wasn't huge before pushing it.
I know review ti
Change by Jay Crotts :
--
keywords: +patch
pull_requests: +7116
stage: -> patch review
___
Python tracker
<https://bugs.python.org/issue33722>
___
___
Python-
New submission from Jay Crotts :
The examples on using mock_open only include instances where objects are mocked
in the REPL, so '__main__'.open is replaced. Commonly objects are mocked for
use in other test modules, so builtins.open would be used instead.
A note about t
Change by Jay Crotts :
--
versions: +Python 3.8 -Python 2.7, Python 3.6, Python 3.7
___
Python tracker
<https://bugs.python.org/issue8243>
___
___
Python-bug
Jay Crotts added the comment:
Thanks Raymond, I wasn't sure if it was a common pattern or not, that makes
sense.
--
___
Python tracker
<https://bugs.python.org/is
Jay Crotts added the comment:
Would it be worth adding an example of unpacking such as, t(**a._asdict()), or
something similar to the documentation ?
--
nosy: +jcrotts
___
Python tracker
<https://bugs.python.org/issue32
Jay Crotts added the comment:
Closing this issue, for the reasons Antoine outlined.
--
resolution: -> rejected
stage: -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.or
Jay Crotts added the comment:
Submitted a PR for this issue, awaiting review.
--
___
Python tracker
<https://bugs.python.org/issue8243>
___
___
Python-bugs-list m
Change by Jay Crotts :
--
keywords: +patch
pull_requests: +5031
stage: needs patch -> patch review
___
Python tracker
<https://bugs.python.org/issue8243>
___
_
Jay Crotts added the comment:
I can create a documentation patch and PR if this still needs doing.
--
nosy: +jcrotts
___
Python tracker
<https://bugs.python.org/issue8
Jay Crotts added the comment:
I think your three reasons make sense, I've only found threading.Timer helpful
in pretty trivial cases.
Do you think a more flexible or efficient Timer class would be useful?
--
___
Python tracker
&
New submission from Jay Crotts :
I wanted to propose the addition of a Timer class to the multiprocessing
library similar to the one that exists in the Threading module.
Timer provides an example of how to extend the Process class that might be
helpful to beginners.
The current lack of a
15 matches
Mail list logo