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/4286#issuecomment-676493029,https://api.github.com/repos/pydata/xarray/issues/4286,676493029,MDEyOklzc3VlQ29tbWVudDY3NjQ5MzAyOQ==,5635139,2020-08-19T15:22:21Z,2020-08-19T15:22:21Z,MEMBER,A Napoleonic PR! (pun intended),"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,668141561
https://github.com/pydata/xarray/pull/4286#issuecomment-676479650,https://api.github.com/repos/pydata/xarray/issues/4286,676479650,MDEyOklzc3VlQ29tbWVudDY3NjQ3OTY1MA==,2448579,2020-08-19T14:59:39Z,2020-08-19T14:59:39Z,MEMBER,Thanks @keewis. Great work!,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,668141561
https://github.com/pydata/xarray/pull/4286#issuecomment-673150613,https://api.github.com/repos/pydata/xarray/issues/4286,673150613,MDEyOklzc3VlQ29tbWVudDY3MzE1MDYxMw==,14808389,2020-08-12T23:00:10Z,2020-08-12T23:00:44Z,MEMBER,"most of the remaining broken links are due to other sections (mostly the `See Also` section). To fix that, `napoleon` has to first add support for preprocessing those sections.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,668141561
https://github.com/pydata/xarray/pull/4286#issuecomment-673079286,https://api.github.com/repos/pydata/xarray/issues/4286,673079286,MDEyOklzc3VlQ29tbWVudDY3MzA3OTI4Ng==,2448579,2020-08-12T19:57:06Z,2020-08-12T19:57:06Z,MEMBER,"@keewis: this is great!
","{""total_count"": 1, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 1, ""rocket"": 0, ""eyes"": 0}",,668141561
https://github.com/pydata/xarray/pull/4286#issuecomment-671049978,https://api.github.com/repos/pydata/xarray/issues/4286,671049978,MDEyOklzc3VlQ29tbWVudDY3MTA0OTk3OA==,14808389,2020-08-09T13:06:49Z,2020-08-09T18:35:11Z,MEMBER,"unfortunately, we won't be able to get this to work until the `defaults` channel includes `sphinx` version 3.2.0: pinning `sphinx` in `doc.yml` doesn't work, RTD will install `sphinx` from `defaults`, even if there's already a `sphinx` version installed. There has already been some [effort](https://github.com/readthedocs/readthedocs.org/pull/5956) to fix that, but I don't know if there was any progress on this since then.
Edit: I guess we could ask support to add a feature flag (`CONDA_APPEND_CORE_REQUIREMENTS`), which might help with that.
Edit: see readthedocs/readthedocs.org#7371","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,668141561
https://github.com/pydata/xarray/pull/4286#issuecomment-670015371,https://api.github.com/repos/pydata/xarray/issues/4286,670015371,MDEyOklzc3VlQ29tbWVudDY3MDAxNTM3MQ==,5635139,2020-08-06T15:54:39Z,2020-08-06T15:54:39Z,MEMBER,Thanks a lot @keewis !,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,668141561
https://github.com/pydata/xarray/pull/4286#issuecomment-667361752,https://api.github.com/repos/pydata/xarray/issues/4286,667361752,MDEyOklzc3VlQ29tbWVudDY2NzM2MTc1Mg==,6628425,2020-07-31T21:21:16Z,2020-07-31T21:21:16Z,MEMBER,"> Anyway, I don't think any of those is part of the public api? If the methods on `CFTimeIndex` are public, I think we either shouldn't reference these classes or make them public (or add a glossary term to define what `CFTimeOffset` means).
Thanks for catching this; I agree they are not currently public API, so let's remove the references to them in docstrings of public functions. And it's true, I think `CFTimeOffset` would be a better name for the base class, but we can wait until later to act on that.
","{""total_count"": 1, ""+1"": 1, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,668141561
https://github.com/pydata/xarray/pull/4286#issuecomment-667355871,https://api.github.com/repos/pydata/xarray/issues/4286,667355871,MDEyOklzc3VlQ29tbWVudDY2NzM1NTg3MQ==,14808389,2020-07-31T21:04:09Z,2020-07-31T21:04:09Z,MEMBER,"@spencerkclark, while working on this PR I got a warning about `CFTimeOffset` not being linkable in `CFTimeIndex.floor` / `ceil` / `round`. Looking through the code, I can only find `BaseCFTimeOffset`, there's no `CFTimeOffset` class (maybe we should rename that base class?).
Anyway, I don't think any of those is part of the public api? If the methods on `CFTimeIndex` are public, I think we either shouldn't reference these classes or make them public (or add a glossary term to define what `CFTimeOffset` means).","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,668141561