home / github / issue_comments

Menu
  • Search all tables
  • GraphQL API

issue_comments: 505555118

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/3042#issuecomment-505555118 https://api.github.com/repos/pydata/xarray/issues/3042 505555118 MDEyOklzc3VlQ29tbWVudDUwNTU1NTExOA== 24736507 2019-06-25T17:59:27Z 2019-06-25T20:30:26Z NONE

Hello @jukent! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

  • In the file xarray/core/dataset.py:

Line 2217:9: F841 local variable 'coord_names' is assigned to but never used Line 2313:5: E303 too many blank lines (2) Line 2344:58: W291 trailing whitespace Line 2349:5: E303 too many blank lines (2) Line 2382:5: E303 too many blank lines (2)

  • In the file xarray/tests/test_dataset.py:

Line 2115:32: E231 missing whitespace after ':' Line 2115:37: E231 missing whitespace after ',' Line 2115:40: E231 missing whitespace after ',' Line 2115:42: E231 missing whitespace after ',' Line 2115:46: E231 missing whitespace after ',' Line 2115:50: E231 missing whitespace after ':' Line 2115:55: E231 missing whitespace after ',' Line 2115:59: E231 missing whitespace after ',' Line 2115:62: E231 missing whitespace after ',' Line 2115:67: E231 missing whitespace after ',' Line 2115:71: E231 missing whitespace after ':' Line 2116:32: E231 missing whitespace after ':' Line 2116:41: E231 missing whitespace after ',' Line 2116:44: E231 missing whitespace after ',' Line 2116:46: E231 missing whitespace after ',' Line 2116:50: E231 missing whitespace after ',' Line 2116:54: E231 missing whitespace after ':' Line 2116:63: E231 missing whitespace after ',' Line 2116:67: E231 missing whitespace after ',' Line 2116:70: E231 missing whitespace after ',' Line 2116:75: E231 missing whitespace after ',' Line 2116:79: E231 missing whitespace after ':' Line 2116:80: E501 line too long (83 > 79 characters) Line 2117:25: E231 missing whitespace after ':' Line 2122:32: E231 missing whitespace after ':' Line 2122:37: E231 missing whitespace after ',' Line 2122:40: E231 missing whitespace after ',' Line 2122:42: E231 missing whitespace after ',' Line 2122:46: E231 missing whitespace after ',' Line 2122:50: E231 missing whitespace after ':' Line 2122:55: E231 missing whitespace after ',' Line 2122:59: E231 missing whitespace after ',' Line 2122:62: E231 missing whitespace after ',' Line 2122:67: E231 missing whitespace after ',' Line 2122:71: E231 missing whitespace after ':' Line 2123:36: E231 missing whitespace after ':' Line 2123:41: E231 missing whitespace after ',' Line 2123:44: E231 missing whitespace after ',' Line 2123:46: E231 missing whitespace after ',' Line 2123:50: E231 missing whitespace after ',' Line 2123:54: E231 missing whitespace after ':' Line 2123:59: E231 missing whitespace after ',' Line 2123:63: E231 missing whitespace after ',' Line 2123:66: E231 missing whitespace after ',' Line 2123:71: E231 missing whitespace after ',' Line 2123:75: E231 missing whitespace after ':' Line 2124:25: E231 missing whitespace after ':'

Comment last updated at 2019-06-25 20:30:26 UTC
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
  460562436
Powered by Datasette · Queries took 76.701ms · About: xarray-datasette