[MacPorts] #62530: pull request CI checks are marked successful even when they fail
MacPorts
noreply at macports.org
Thu Mar 25 05:58:27 UTC 2021
#62530: pull request CI checks are marked successful even when they fail
----------------------------+---------------------
Reporter: ryandesign | Owner: admin@…
Type: defect | Status: new
Priority: Normal | Milestone:
Component: buildbot/mpbb | Version:
Resolution: | Keywords:
Port: |
----------------------------+---------------------
Comment (by mojca):
I recently made a PR with tons of different python packages (just for the
sake of adding py39 subport) in addition to the main package that switched
from python 3.8 to 3.9. I nearly oversaw the fact that some packages
failed to build, and the log was so long that it was basically impossible
to figure out what built and what didn't. I only tested some of the
packages, but everything was green, so I felt comfortable merging and
almost did that, I only later noticed livecheck warnings.
I fail to understand why it's better to deliberately mark a build a
successful. If a dependency fails, the main port won't even be checked. At
all. And yet it will be shown in green. If the build fails and is marked
with the red color, it's still straightforward to check why the build fail
and deliberately decide that it's still ok to merge something. It's not
like we prevent merging failed builds.
Combined with the fact that we are no longer receiving emails with build
failures, that's a perfect recipe to never notice a failed build: neither
before nor after the merge.
--
Ticket URL: <https://trac.macports.org/ticket/62530#comment:3>
MacPorts <https://www.macports.org/>
Ports system for macOS
More information about the macports-tickets
mailing list