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/5188#issuecomment-822685095,https://api.github.com/repos/pydata/xarray/issues/5188,822685095,MDEyOklzc3VlQ29tbWVudDgyMjY4NTA5NQ==,226037,2021-04-19T18:33:23Z,2021-04-19T18:34:12Z,MEMBER,"> I would do another regex special to check whether it contains a `\` @max-sixty I was thinking to something more brutal: I would drop the `r`-prefix everywhere and fix by hand the failing tests (if any) ;)","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,860909006 https://github.com/pydata/xarray/pull/5188#issuecomment-822677989,https://api.github.com/repos/pydata/xarray/issues/5188,822677989,MDEyOklzc3VlQ29tbWVudDgyMjY3Nzk4OQ==,226037,2021-04-19T18:21:59Z,2021-04-19T18:21:59Z,MEMBER,"@max-sixty I like the cleanup even it it is rather big. I for one find it very confusing when tests use non standard helpers. BTW, I think you can skip the `r`-prefix everywhere making the change even more appealing. It is only useful when using `\` operands in the the match string which is almost never the case in xarray.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,860909006