html_url,issue_url,id,node_id,user,created_at,updated_at,author_association,body,reactions,performed_via_github_app,issue
https://github.com/pydata/xarray/pull/5678#issuecomment-902430683,https://api.github.com/repos/pydata/xarray/issues/5678,902430683,IC_kwDOAMm_X841yf_b,14371165,2021-08-20T04:57:29Z,2021-08-20T04:57:29Z,MEMBER,"Yes! Literal and TypeGuard seems like great ways of narrowing down types without using slow isinstance checks everywhere.
Some nice reading:
https://adamj.eu/tech/2021/05/17/python-type-hints-how-to-narrow-types-with-isinstance-assert-literal/
https://adamj.eu/tech/2021/06/09/python-type-hints-how-to-narrow-types-with-typeguard/","{""total_count"": 1, ""+1"": 1, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,962647570
https://github.com/pydata/xarray/pull/5678#issuecomment-902014311,https://api.github.com/repos/pydata/xarray/issues/5678,902014311,IC_kwDOAMm_X841w6Vn,14371165,2021-08-19T15:32:58Z,2021-08-19T15:32:58Z,MEMBER,"I think this is ready for review now.
The only thing missing now is a typing for positive-only ints. I haven't figured out how to do that though.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,962647570
https://github.com/pydata/xarray/pull/5678#issuecomment-894540803,https://api.github.com/repos/pydata/xarray/issues/5678,894540803,IC_kwDOAMm_X841UZwD,14371165,2021-08-06T22:03:32Z,2021-08-06T22:03:32Z,MEMBER,"I found `TypedDict` in `typing_extensions` as well. So now I think the mypy should behave the same for all python versions.
It got quite messy because `typing_extensions` isnt a dependency of xarray, see #5495.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,962647570
https://github.com/pydata/xarray/pull/5678#issuecomment-894483982,https://api.github.com/repos/pydata/xarray/issues/5678,894483982,IC_kwDOAMm_X841UL4O,14371165,2021-08-06T19:50:12Z,2021-08-06T19:53:15Z,MEMBER,"@max-sixty is there a way turn off the typing, but still having T_Options defined? I think having no typing is better than adding typing in the 3.7 case.
Maybe I should just wrap it inside the if check as well?","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,962647570
https://github.com/pydata/xarray/pull/5678#issuecomment-894434509,https://api.github.com/repos/pydata/xarray/issues/5678,894434509,IC_kwDOAMm_X841T_zN,14371165,2021-08-06T18:15:37Z,2021-08-06T18:15:37Z,MEMBER,"I'm a little uncertain about the 3.7 solution and what happens once we start using these typings more (for example is `sum(int, Any)` ok?).
But we can drop 3.7 in december I think so maybe these typing inconsistencies is fine?","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,962647570