Zulip Chat Archive

Stream: mathlib4

Topic: CI failure in import-graph


Kim Morrison (Sep 14 2025 at 22:53):

@Jon Eugster @Paul Lezeau, just a heads up that "feat: add #import_diff command #73" https://github.com/leanprover-community/import-graph/pull/73 seems to have introduced a failure in CI (perhaps intermittent)?

See https://github.com/leanprover-community/import-graph/actions/runs/17717638873/job/50344747854?pr=81.

This hit me when trying to upgrade the toolchain today.

Kim Morrison (Sep 14 2025 at 22:54):

This is non-deterministic, and whether we see an error depends on lake's choice of build order, which we can't rely on.

Kim Morrison (Sep 14 2025 at 22:56):

As I'm in the middle of a release process, I don't have time to stop to fix this, so I am deleting the two files that cause the non-determinism, and asking if you two can arrange fixing this after the fact? Thank you. :-)

Paul Lezeau (Sep 14 2025 at 22:59):

Sure! I’ll take a look at this tomorrow:)

Jon Eugster (Sep 15 2025 at 15:18):

Thanks for spotting Kim! Do I see this correctly that there is no CI running on PRs to that repo? I'll have a look and fix CI if required to avoid this in the future

Jon Eugster (Sep 15 2025 at 20:19):

import-graph#83 should fix this by removing any non-determinism and (hopefully?) fixing CI to run on all PRs. @Paul Lezeau I've also sorted the outputs of the command alphabetically

Paul Lezeau (Sep 15 2025 at 20:50):

Oh nice! Thanks!


Last updated: Dec 20 2025 at 21:32 UTC