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/2553#issuecomment-437578022,https://api.github.com/repos/pydata/xarray/issues/2553,437578022,MDEyOklzc3VlQ29tbWVudDQzNzU3ODAyMg==,24736507,2018-11-10T11:40:54Z,2018-12-12T00:54:59Z,NONE,"Hello @TomNicholas! Thanks for updating the PR. - In the file [`xarray/backends/api.py`](https://github.com/TomNicholas/xarray/blob/ebbe47f450ed4407655bd9a4ed45274b140452dd/xarray/backends/api.py), following are the PEP8 issues : > [Line 647:80](https://github.com/TomNicholas/xarray/blob/ebbe47f450ed4407655bd9a4ed45274b140452dd/xarray/backends/api.py#L647): [E501](https://duckduckgo.com/?q=pep8%20E501) line too long (81 > 79 characters) - In the file [`xarray/core/combine.py`](https://github.com/TomNicholas/xarray/blob/ebbe47f450ed4407655bd9a4ed45274b140452dd/xarray/core/combine.py), following are the PEP8 issues : > [Line 381:36](https://github.com/TomNicholas/xarray/blob/ebbe47f450ed4407655bd9a4ed45274b140452dd/xarray/core/combine.py#L381): [E226](https://duckduckgo.com/?q=pep8%20E226) missing whitespace around arithmetic operator > [Line 536:75](https://github.com/TomNicholas/xarray/blob/ebbe47f450ed4407655bd9a4ed45274b140452dd/xarray/core/combine.py#L536): [E211](https://duckduckgo.com/?q=pep8%20E211) whitespace before '(' > [Line 537:57](https://github.com/TomNicholas/xarray/blob/ebbe47f450ed4407655bd9a4ed45274b140452dd/xarray/core/combine.py#L537): [E126](https://duckduckgo.com/?q=pep8%20E126) continuation line over-indented for hanging indent - In the file [`xarray/testing.py`](https://github.com/TomNicholas/xarray/blob/ebbe47f450ed4407655bd9a4ed45274b140452dd/xarray/testing.py), following are the PEP8 issues : > [Line 148:1](https://github.com/TomNicholas/xarray/blob/ebbe47f450ed4407655bd9a4ed45274b140452dd/xarray/testing.py#L148): [W391](https://duckduckgo.com/?q=pep8%20W391) blank line at end of file - There are no PEP8 issues in the file [`xarray/tests/__init__.py`](https://github.com/TomNicholas/xarray/blob/ebbe47f450ed4407655bd9a4ed45274b140452dd/xarray/tests/__init__.py) ! - In the file [`xarray/tests/test_backends.py`](https://github.com/TomNicholas/xarray/blob/ebbe47f450ed4407655bd9a4ed45274b140452dd/xarray/tests/test_backends.py), following are the PEP8 issues : > [Line 2165:36](https://github.com/TomNicholas/xarray/blob/ebbe47f450ed4407655bd9a4ed45274b140452dd/xarray/tests/test_backends.py#L2165): [E127](https://duckduckgo.com/?q=pep8%20E127) continuation line over-indented for visual indent > [Line 2172:36](https://github.com/TomNicholas/xarray/blob/ebbe47f450ed4407655bd9a4ed45274b140452dd/xarray/tests/test_backends.py#L2172): [E126](https://duckduckgo.com/?q=pep8%20E126) continuation line over-indented for hanging indent - In the file [`xarray/tests/test_combine.py`](https://github.com/TomNicholas/xarray/blob/ebbe47f450ed4407655bd9a4ed45274b140452dd/xarray/tests/test_combine.py), following are the PEP8 issues : > [Line 495:65](https://github.com/TomNicholas/xarray/blob/ebbe47f450ed4407655bd9a4ed45274b140452dd/xarray/tests/test_combine.py#L495): [E211](https://duckduckgo.com/?q=pep8%20E211) whitespace before '(' > [Line 523:52](https://github.com/TomNicholas/xarray/blob/ebbe47f450ed4407655bd9a4ed45274b140452dd/xarray/tests/test_combine.py#L523): [E231](https://duckduckgo.com/?q=pep8%20E231) missing whitespace after ',' > [Line 523:54](https://github.com/TomNicholas/xarray/blob/ebbe47f450ed4407655bd9a4ed45274b140452dd/xarray/tests/test_combine.py#L523): [E231](https://duckduckgo.com/?q=pep8%20E231) missing whitespace after ',' > [Line 523:61](https://github.com/TomNicholas/xarray/blob/ebbe47f450ed4407655bd9a4ed45274b140452dd/xarray/tests/test_combine.py#L523): [E231](https://duckduckgo.com/?q=pep8%20E231) missing whitespace after ',' > [Line 524:57](https://github.com/TomNicholas/xarray/blob/ebbe47f450ed4407655bd9a4ed45274b140452dd/xarray/tests/test_combine.py#L524): [E241](https://duckduckgo.com/?q=pep8%20E241) multiple spaces after ',' > [Line 525:55](https://github.com/TomNicholas/xarray/blob/ebbe47f450ed4407655bd9a4ed45274b140452dd/xarray/tests/test_combine.py#L525): [E241](https://duckduckgo.com/?q=pep8%20E241) multiple spaces after ',' > [Line 526:55](https://github.com/TomNicholas/xarray/blob/ebbe47f450ed4407655bd9a4ed45274b140452dd/xarray/tests/test_combine.py#L526): [E241](https://duckduckgo.com/?q=pep8%20E241) multiple spaces after ',' > [Line 527:57](https://github.com/TomNicholas/xarray/blob/ebbe47f450ed4407655bd9a4ed45274b140452dd/xarray/tests/test_combine.py#L527): [E241](https://duckduckgo.com/?q=pep8%20E241) multiple spaces after ',' > [Line 597:52](https://github.com/TomNicholas/xarray/blob/ebbe47f450ed4407655bd9a4ed45274b140452dd/xarray/tests/test_combine.py#L597): [E203](https://duckduckgo.com/?q=pep8%20E203) whitespace before ',' > [Line 646:80](https://github.com/TomNicholas/xarray/blob/ebbe47f450ed4407655bd9a4ed45274b140452dd/xarray/tests/test_combine.py#L646): [E501](https://duckduckgo.com/?q=pep8%20E501) line too long (85 > 79 characters) > [Line 647:80](https://github.com/TomNicholas/xarray/blob/ebbe47f450ed4407655bd9a4ed45274b140452dd/xarray/tests/test_combine.py#L647): [E501](https://duckduckgo.com/?q=pep8%20E501) line too long (85 > 79 characters) > [Line 658:80](https://github.com/TomNicholas/xarray/blob/ebbe47f450ed4407655bd9a4ed45274b140452dd/xarray/tests/test_combine.py#L658): [E501](https://duckduckgo.com/?q=pep8%20E501) line too long (83 > 79 characters) > [Line 659:80](https://github.com/TomNicholas/xarray/blob/ebbe47f450ed4407655bd9a4ed45274b140452dd/xarray/tests/test_combine.py#L659): [E501](https://duckduckgo.com/?q=pep8%20E501) line too long (87 > 79 characters) ##### Comment last updated on December 12, 2018 at 00:54 Hours UTC","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,379415229