[Python-Dev] Merge request reminder.

2021-03-23 Thread Alfred Perlstein
Hello, a while back we hit a bug with doctest and mock.call that confused a few people on our team for a while. I dug into fixing it and made doctest more defensive around un-unwrappable calls. I submitted a PR (https://github.com/python/cpython/pull/22981) for an Issue

[Python-Dev] Doctest crashes on unittest.call import. bpo-35753

2021-02-21 Thread Alfred Perlstein
tive patch to be reviewed. thanks, -Alfred Forwarded Message Subject: [Python-Dev] minor PR 22981 waiting review ~20 days: https://github.com/python/cpython/pull/22981 Date: Mon, 14 Dec 2020 13:25:36 -0800 From: Alfred Perlstein Organization: FreeBSD To: Python Dev

[Python-Dev] Doctest does not work with mock.call PR #22981

2021-01-21 Thread Alfred Perlstein
Hello, There's been a bug open where doctest can break if there are proxy objects that fail to unwrap (https://bugs.python.org/issue35753) since python 3.7, this includes when importing 'call' from the 'mock' module. Does someone have time to review PR 22981

[Python-Dev] minor PR 22981 waiting review ~20 days: https://github.com/python/cpython/pull/22981

2020-12-14 Thread Alfred Perlstein
PR 22981 is a minor update to doctest to allow it to safely consume modules which contain objects that cause inspect.unwrap to throw. I believe the review comments in the PR have been addressed at this point for ~20 days.  The patch is relatively small, reviews are done, but it is unmerged.

[Python-Dev] Re: Ideas for improving the contribution experience

2020-12-07 Thread Alfred Perlstein
On 10/16/20 3:29 PM, Tal Einat wrote: (Context: Continuing to prepare for the core dev sprint next week. Since the sprint is near, *I'd greatly appreciate any quick comments, feedback and ideas!*) Following up my collection of past beginning contributor experiences, I've collected these

[Python-Dev] Re: review for doctest fix: bpo-35753

2020-11-25 Thread Alfred Perlstein
On 11/6/20 6:13 PM, Steven D'Aprano wrote: For the benefit of others, the problem is that `unittest.mock.call.__wrapped__` generates a new object, which in turn has a dynamic `__wrapped__` attribute, which does the same, thus generating an infinite chain of *distinct* proxies. Being distinct

[Python-Dev] review for doctest fix: bpo-35753

2020-11-06 Thread Alfred Perlstein
Hello, We are using doctest to give our developers easy access to write very fast unit tests and encourage "tests as documentation". Recently we hit an issue where doctest crashes on certain objects that fail to "unwrap". Looking at the code and reasoning behind the crash it seems like we

Re: [Python-Dev] Help preventing SIGPIPE/SIG_DFL anti-pattern.

2018-06-30 Thread Alfred Perlstein
On 6/30/18 4:20 PM, Greg Ewing wrote: Alfred Perlstein wrote: I am asking if there's a way we can discourage the use of "signal(SIGPIPE, SIG_DFL)" unless the user really understands what they are doing. Maybe there's some way that SIGPIPEs on stdout could be handled differently

[Python-Dev] Help preventing SIGPIPE/SIG_DFL anti-pattern.

2018-06-30 Thread Alfred Perlstein
Hello, I'm looking for someone in the python community to help with a problem of anti-patterns showing up dealing with SIGPIPE. Specifically I've noticed an anti-pattern developing where folks will try to suppress broken pipe errors written to stdout by setting SIGPIPE's disposition to

[Python-Dev] Help preventing SIGPIPE/SIG_DFL anti-pattern.

2018-06-30 Thread Alfred Perlstein
(sorry for the double post, looks like maybe attachments are dropped, inlined the attachment this time.) Hello, I'm looking for someone in the python community to help with a problem of anti-patterns showing up dealing with SIGPIPE. Specifically I've noticed an anti-pattern developing where