Stephen Morton

This would be a big help for me! Previous versions of NNW would change the font on the review to a "visited link" color which was helpful when I wanted...

The error moved to `mypy/checker.py` - there's a lot of places in that file with the names of typing.* classes as string literals passed to `TypeChecker.named_generic_type`. It works if `'_collections_abc.Sequence'`...

Seems like a lot of the diff in diff shades only have one method call. This in particular seems especially bad: ```diff assert ( makeFY5253NearestEndMonthQuarter( weekday=1, startingMonth=3, qtr_with_extra_week=3 - ).freqstr...

I tested preview on a code base of mine, and it mostly looked good. The only change I was really surprised by was from string processing; the same issue as...

If you want to write tests against the S3 API, Moto is very good: https://github.com/getmoto/moto A version modified to emulate IA's incompatibilities shouldn't be that hard to make, although maintenance...

Currently we have _typeshed. What about expanding that? I'd suggest that for a given stdlib module foo we should import from `_typeshed.foo` while for a third-party module bar we should...

I'm making this a draft pending discussion in https://github.com/python/typeshed/issues/11141 I'm inclined to think using an explicit `__module__` attribute (like runtime does) is a better way to handle this discrepancy.

I think what I mean is that it's not obvious what "follow the implementation" means here, because different parts of the implementation are conflicting with each other. If we have:...

I made several MRs along these lines, but then I discovered that the _pydecimal module changes its `__name__` attribute, which I hadn't though of as an option: https://github.com/python/cpython/blob/d91e43ed7839b601cbeadd137d89e69e2cc3efda/Lib/_pydecimal.py#L151 Which got...

As it turned out, classes which set their own `__module__` attribute were a lot easier to handle than modules which set their own `__name__` attribute. I have a draft implementation...