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/2467#issuecomment-427582505,https://api.github.com/repos/pydata/xarray/issues/2467,427582505,MDEyOklzc3VlQ29tbWVudDQyNzU4MjUwNQ==,5635139,2018-10-06T15:24:45Z,2018-10-06T15:24:45Z,MEMBER,"I added a commit to clear up some of the issues that @shoyer found. Assuming tests pass, I'll merge shortly given risk of conflicts - please lmk any final comments. I'm also happy to do any flups after merge this weekend.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,367410303 https://github.com/pydata/xarray/pull/2467#issuecomment-427535714,https://api.github.com/repos/pydata/xarray/issues/2467,427535714,MDEyOklzc3VlQ29tbWVudDQyNzUzNTcxNA==,5635139,2018-10-06T01:25:22Z,2018-10-06T01:25:22Z,MEMBER,Is @pep8speaks respecting `# noqa` directions? It's still complaining about ambiguous variable names here: https://github.com/max-sixty/xarray/blob/pytest/xarray/tests/test_indexing.py#L27,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,367410303