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/4599#issuecomment-732331278,https://api.github.com/repos/pydata/xarray/issues/4599,732331278,MDEyOklzc3VlQ29tbWVudDczMjMzMTI3OA==,14808389,2020-11-23T18:05:31Z,2020-11-23T18:11:49Z,MEMBER,"That's somewhat different, I think. Branch protection defines CI that has to pass in order to be able to merge the PR, while allowed failures change the result of the CI. For the current implementation of this, see the changes in #4584.
Edit: the `continue-on-error` setting will do the same, except the job passes silently which is not ideal","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,748264086
https://github.com/pydata/xarray/pull/4599#issuecomment-732284031,https://api.github.com/repos/pydata/xarray/issues/4599,732284031,MDEyOklzc3VlQ29tbWVudDczMjI4NDAzMQ==,14808389,2020-11-23T16:47:46Z,2020-11-23T16:47:46Z,MEMBER,"If I remember correctly, we decided to mark the upstream-dev CI as allowed failure. If there's a way to mark the PR CI as such, I think the only benefit is that a allowed failure of the old azure CI is a bit more visible (see https://github.com/pydata/xarray/pull/4584#issuecomment-729726028).","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,748264086
https://github.com/pydata/xarray/pull/4599#issuecomment-732275375,https://api.github.com/repos/pydata/xarray/issues/4599,732275375,MDEyOklzc3VlQ29tbWVudDczMjI3NTM3NQ==,14808389,2020-11-23T16:33:19Z,2020-11-23T16:33:19Z,MEMBER,"great, thanks again, @andersy005. Let's merge now and leave the choice between the old and new PR CI to a different PR.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,748264086