rust-libp2p icon indicating copy to clipboard operation
rust-libp2p copied to clipboard

mergify continously approving approved pull requests

Open mxinden opened this issue 1 year ago • 6 comments

Summary

Mergify seems to continously approve already approved pull requests. See e.g. https://github.com/libp2p/rust-libp2p/pull/5206.

Expected behavior

Approve once.

Actual behavior

Approve ~ every mintue.

Relevant log output

No response

Possible Solution

No response

Version

No response

Would you like to work on fixing this bug ?

No

mxinden avatar Mar 04 '24 12:03 mxinden

I paused all merge queues, thus far without success.

https://dashboard.mergify.com/github/libp2p/repo/rust-libp2p/queues?branch=master&queues=default

mxinden avatar Mar 04 '24 12:03 mxinden

Potential hot-fix: https://github.com/libp2p/rust-libp2p/pull/5208

mxinden avatar Mar 04 '24 12:03 mxinden

#5208 did the trick. I unpaused the merge queue.

mxinden avatar Mar 04 '24 12:03 mxinden

thanks Max, it seems mergify approves but doesn't add the send-it label right? With your hotfix we now need to manually approve the deps PR's from dependabot right?

jxs avatar Mar 04 '24 16:03 jxs

thanks Max, it seems mergify approves but doesn't add the send-it label right?

Did it ever? As far as I can tell, it simply enqueues it right away:

https://github.com/libp2p/rust-libp2p/blob/46ec6c416a9b06c089236a8d6d44e55d0c3f7edd/.github/mergify.yml#L36-L42

mxinden avatar Mar 04 '24 18:03 mxinden

With your hotfix we now need to manually approve the deps PR's from dependabot right?

I don't know what the best process is. I don't think the PRs are urgent. Instead I suggest debugging the mergify issue first.

mxinden avatar Mar 04 '24 18:03 mxinden