issue_comments: 897396180

This data as json

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/5690#issuecomment-897396180 https://api.github.com/repos/pydata/xarray/issues/5690 897396180 IC_kwDOAMm_X841fS3U 1217238 2021-08-12T06:57:56Z 2021-08-12T06:57:56Z MEMBER

I would suggest that new tests use -> None: so mypy checks them, and we build up some tests for our typing through that — any thoughts?

I'm not opposed to suggesting -> None for tests, but I would hesitate to require it: - It's not unusual for tests to be incompatible with type checking, e.g., because they are checking invalid input, or rely on tricks like mocks. - It's also a fair amount of boilerplate to add to code that already has a lot of boilerplate (for things like parametrized tests & fixtures), so readability could suffer.

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   965939431