Closed kyleam closed 8 months ago
Was just gona do this - thanks for taking care of it
An additional reason for pulling it out is that it'll let us see whether Drone is still misbehaving
Hmm, no luck troubleshooting that. (Toggled off "require status check" for these with intent of toggling it right back on to see if the old off/on resolved it, but now the checks aren't event showing up as an option.)
Anyway, no reason to hold this up to debug more. We can open scratch PRs if needed.
Here are the green drone runs:
Will merge (I can't officially approve this since I opened the PR).
This brings over the @barrettk's initial commit from gh-610, which updates the MPN snapshot to one with nmrec. As discussed here, not doing so in gh-606 was an oversight, and it'd be good to get that into main without tying it up to an unrelated PR.
An additional reason for pulling it out is that it'll let us see whether Drone is still misbehaving: https://github.com/metrumresearchgroup/bbr/pull/606#issuecomment-1756226915