home / github / issue_comments

Menu
  • GraphQL API
  • Search all tables

issue_comments: 239215093

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/917#issuecomment-239215093 https://api.github.com/repos/pydata/xarray/issues/917 239215093 MDEyOklzc3VlQ29tbWVudDIzOTIxNTA5Mw== 1217238 2016-08-11T16:30:17Z 2016-08-11T16:30:17Z MEMBER

I have a few minor suggestions, but otherwise this looks very nice!

To ensure that the documentation subpages are built for these methods (which makes the sphinx link work), you need to add them to doc/api.rst. These should go in the IO / Conversion section.

If you haven't tested the docs locally with make html, take a look at the development version of the docs (http://xarray.pydata.org/en/latest/) about 5-10 minutes after I merge this to make sure everything looks right. RST can be tricky to get right.

{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
  167091064
Powered by Datasette · Queries took 0.853ms · About: xarray-datasette