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/5776#issuecomment-927232092,https://api.github.com/repos/pydata/xarray/issues/5776,927232092,IC_kwDOAMm_X843RHBc,23625371,2021-09-26T04:59:24Z,2021-09-26T04:59:24Z,CONTRIBUTOR,"> The only other thing this needs is a test to check that the error is raised with the correct message (i.e. `pytest.raises(TypeError, match=""may fix this issue"")`
I believe that's covered by https://github.com/pydata/xarray/blob/ec2657bfc1a426aa3559243b02ecf48de6673506/xarray/tests/test_combine.py#L1109 (i.e. no change is needed to the existing code)","{""total_count"": 1, ""+1"": 1, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,990509161
https://github.com/pydata/xarray/pull/5776#issuecomment-927108350,https://api.github.com/repos/pydata/xarray/issues/5776,927108350,IC_kwDOAMm_X843Qoz-,23625371,2021-09-25T11:35:37Z,2021-09-25T11:35:37Z,CONTRIBUTOR,"Alrighty updated, see what you think","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,990509161
https://github.com/pydata/xarray/pull/5776#issuecomment-914762048,https://api.github.com/repos/pydata/xarray/issues/5776,914762048,IC_kwDOAMm_X842hilA,23625371,2021-09-08T01:13:06Z,2021-09-08T01:13:06Z,CONTRIBUTOR,"We could try and tighten this up in a few ways (I'm not sure how much effort it is worth):
- only talk about cf_time if the error is in the time dimension (or if one of the types is `cftime.datetime`)
- only talk about cf_time if the function which tried to do the combining actually has this argument (e.g. `combine_by_coords` doesn't have `use_cftime`, only `open_mfdataset` does so the current implementation could be confusing), to do this I'd use a try except block further up the stack rather than altering the message in `_ensure_same_types`
Thoughts @mathause and @TomNicholas ?","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,990509161