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/5334#issuecomment-873140890,https://api.github.com/repos/pydata/xarray/issues/5334,873140890,MDEyOklzc3VlQ29tbWVudDg3MzE0MDg5MA==,11541317,2021-07-02T17:08:58Z,2021-07-02T17:09:04Z,CONTRIBUTOR,"Thanks @dcherian! I really liked the PR process in xarray, having 3 reviewers here was awesome and a crucial part for building quality code. Thanks for your hard work and comments! Hope this to be the first PR of many more!","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,894753913
https://github.com/pydata/xarray/pull/5334#issuecomment-872968758,https://api.github.com/repos/pydata/xarray/issues/5334,872968758,MDEyOklzc3VlQ29tbWVudDg3Mjk2ODc1OA==,2448579,2021-07-02T12:42:31Z,2021-07-02T12:42:31Z,MEMBER,"I think we can address any concerns about test fixtures in a future PR.

Thanks @santisoler. I see this is your first contribution. welcome to xarray!","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,894753913
https://github.com/pydata/xarray/pull/5334#issuecomment-856195015,https://api.github.com/repos/pydata/xarray/issues/5334,856195015,MDEyOklzc3VlQ29tbWVudDg1NjE5NTAxNQ==,11541317,2021-06-07T19:21:54Z,2021-06-07T19:21:54Z,CONTRIBUTOR,"@shoyer I've just fixed some errors on the `TestAxesKwargs` tests that were using logscales without passing any coordinates.
I've also added an error raising if the passed coordinates have at least one non-positive element.
Hope CIs pass now. Let me know if you feel that something else should be changed.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,894753913
https://github.com/pydata/xarray/pull/5334#issuecomment-846034540,https://api.github.com/repos/pydata/xarray/issues/5334,846034540,MDEyOklzc3VlQ29tbWVudDg0NjAzNDU0MA==,11541317,2021-05-21T15:28:25Z,2021-05-21T15:28:25Z,CONTRIBUTOR,"@shoyer Now I think this PR is ready to be reviewed. I added one test function for the `_infer_interval_breaks` private function with the optional `scale=""log""` argument. I've also added a new test class for checking if calling `pcolormesh` with both `xscale=""log""` and `yscale=""log""` works as expected. I added the bug fix to `whats-new.rst` and run `pre-commit run --all-files` locally with success. I don't think we need to change anything in `api.rst`.

Let me know if you think I should change or add anything else, and feel free to make any suggestion on the review.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,894753913
https://github.com/pydata/xarray/pull/5334#issuecomment-845921245,https://api.github.com/repos/pydata/xarray/issues/5334,845921245,MDEyOklzc3VlQ29tbWVudDg0NTkyMTI0NQ==,11541317,2021-05-21T12:38:06Z,2021-05-21T12:38:06Z,CONTRIBUTOR,"Thanks @shoyer!

> could you please look into adding a unit-test, to ensure that this stays fixed?

Yes! I got a little bit busier after I opened this PR and couldn't find the time to do so. Let me find a gap to add some tests. I'll ping you as soon as the PR is ready for a review. Thanks!","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,894753913
https://github.com/pydata/xarray/pull/5334#issuecomment-845605670,https://api.github.com/repos/pydata/xarray/issues/5334,845605670,MDEyOklzc3VlQ29tbWVudDg0NTYwNTY3MA==,1217238,2021-05-21T02:26:18Z,2021-05-21T02:26:18Z,MEMBER,"this is great, thank you!

could you please look into adding a unit-test, to ensure that this stays fixed?

see http://xarray.pydata.org/en/stable/contributing.html#writing-tests","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,894753913