issue_comments: 854047989

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/5425#issuecomment-854047989 https://api.github.com/repos/pydata/xarray/issues/5425 854047989 MDEyOklzc3VlQ29tbWVudDg1NDA0Nzk4OQ== 5635139 2021-06-03T17:25:18Z 2021-06-03T17:25:18Z MEMBER

Ha, no problem. And don't worry about the commit quality — it's all squashed into one at the end. I generally write one good message at the start but that's all.

This looks good now!

One small change — could we say in the error message that they all need to be either all DataArrays or all Datasets? For a new user, the currently proposed messages could suggest that they all need to be the type in the message, and that type is determined by the order of the args, which is a bit confusing. That might mean the error message is the same for both, or only slightly different. Does that make sense?

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   908651787