Zulip Chat Archive
Stream: new members
Topic: build failure
Mark Gerads (Mar 01 2021 at 20:17):
It says
Changes approved
and Mathlib is being rebuilt again. Does this mean I don't need to do anything?
Eric Wieser (Mar 01 2021 at 20:18):
If bors says it's retrying, then you don't need to do anything
Eric Wieser (Mar 01 2021 at 20:19):
Although it's good practice to look at the logs to guess whether it's your fault
Bryan Gin-ge Chen (Mar 01 2021 at 20:19):
You pushed a new commit just now, so your PR was taken out of the queue. I've "delegated" the PR to you with bors d+
, so you can put it back on the queue whenever you want.
Mark Gerads (Mar 01 2021 at 20:21):
I'll see what that means I need to do after it shows Mathlib as successfully built.
Bryan Gin-ge Chen (Mar 01 2021 at 20:23):
(All you need to do is put bors r+
on its own line in a GitHub comment.)
Eric Wieser (Mar 01 2021 at 20:25):
Note that bors failing doesn't even mean a conflict has appeared in master - it just means that when you combine the pile of PRs into a single build on top of master, they either conflict with each other or with master
Mark Gerads (Mar 01 2021 at 20:30):
Is it okay for me to merge it now then? I figure I should wait; 'tis more proper. I am excited to see this contribution in Mathlib.
Bryan Gin-ge Chen (Mar 01 2021 at 20:35):
Sure, I think it's safe enough! It'll join the other PRs in queue: https://app.bors.tech/repositories/24316
Last updated: Dec 20 2023 at 11:08 UTC