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/issues/5755#issuecomment-910467901,https://api.github.com/repos/pydata/xarray/issues/5755,910467901,IC_kwDOAMm_X842RKM9,5635139,2021-09-01T16:50:56Z,2021-09-01T16:50:56Z,MEMBER,"> What I wonder about is why we called it `T_Dataset` instead of `T_DatasetOpsMixin`?
> Is it a trick to make it look good in docs or?

The goal was for the type that's used throughout Xarray to be called `T_Dataset`, but not sure why that means we have to change the definition — I don't think there's a good reason.

Should we change that back, and change the mypy file files definition to `.py[i]`?","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,985162305
https://github.com/pydata/xarray/issues/5755#issuecomment-910464246,https://api.github.com/repos/pydata/xarray/issues/5755,910464246,IC_kwDOAMm_X842RJT2,5635139,2021-09-01T16:46:03Z,2021-09-01T16:47:01Z,MEMBER,"Edit: @Illviljan and I commented at the same time, he seems to have found the error!

---
Great spot @benbovy . I think what's happening is that the standard run only checks `.py` files, rather than `.pyi` files. So `mypy .` collects it, but pre-commit and `mypy` doesn't.

ref https://github.com/pre-commit/mirrors-mypy/issues/33 for the pre-commit issue.

On the errors themselves — @rhkleijn I think I have upset your magnum opus despite the best of intentions. If you're up for having a look, that would be great. Otherwise we could try and revert my original PR, though I worry this sort of refactor is difficult to revert.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,985162305