Add condition attribute for unresolved PR comments

[melink14]

My org has a style of leaving final 'nit' comments and approving assuming they'll be fixed before merge but it seems there's no way to tell mergify to not merge if there are open comments.

The alternative would be to create a status check for open comments but it seems round about. :)

Thanks!

[jd]

Hi @melink14!

How do you recognize "open comments"?

[melink14]

Thanks for the reply.

I mean resolved vs unresolved in github parlance:

I was thinking that resolved might be the same as what's referred to by the dismissed-reviews-by but not 100% sure.

[jd]

They are not the same indeed, the dismissed-reviews-by or commented-reviews-by attributes are for the general review of the PR, not about individuals comments.

[melink14]

GitHub recently released branch protection against unresolved comments so this should work directly for users rely on them (I do).

(Of course, this is only after hours spent on a custom workflow...)