From 2f58d13c2bc5ef8689d08f94bd11286b31768342 Mon Sep 17 00:00:00 2001 From: DROOdotFOO Date: Mon, 13 Jul 2026 20:15:29 +0200 Subject: [PATCH] ci: tolerate upstream mix.lock drift CI's `mix deps.get --check-locked` fast-fails whenever any transitive dep publishes a newer compatible patch after the lock was committed -- a recurring false failure that reds every PR (and master) within minutes of an upstream release (ex_ast, then phoenix_live_view, ...). Relax the two ci-unified.yml install steps to `mix deps.get --check-locked || mix deps.get`: prefer the committed lock, but fall back to a plain resolve on pure upstream drift instead of failing the pipeline. The pre-commit hook and `mix raxol.check` still guard genuinely uncommitted lock changes locally. Also refresh the lock to current (phoenix_live_view 1.2.6 -> 1.2.7) so the common path stays reproducible. --- .github/workflows/ci-unified.yml | 22 ++++++++++++---------- mix.lock | 2 +- 2 files changed, 13 insertions(+), 11 deletions(-) diff --git a/.github/workflows/ci-unified.yml b/.github/workflows/ci-unified.yml index 811e099ef..667b14e58 100644 --- a/.github/workflows/ci-unified.yml +++ b/.github/workflows/ci-unified.yml @@ -117,11 +117,12 @@ jobs: mix local.hex --force mix local.rebar --force mix deps.clean --unused - # --check-locked fails fast if mix.exs was edited without committing - # an updated mix.lock. Catches the failure mode where new deps land - # without their lockfile entries and surface as confusing compile - # errors downstream. - mix deps.get --check-locked + # Prefer the committed lock (fast-fails if mix.exs was edited without + # a committed mix.lock). Fall back to a plain resolve so pure upstream + # drift -- a transitive dep publishing a newer patch after the lock was + # committed -- does not red the whole pipeline. The pre-commit hook and + # `mix raxol.check` still guard genuinely uncommitted lock changes. + mix deps.get --check-locked || mix deps.get mix deps.compile # ======================================== @@ -507,11 +508,12 @@ jobs: mix local.hex --force mix local.rebar --force mix deps.clean --unused - # --check-locked fails fast if mix.exs was edited without committing - # an updated mix.lock. Catches the failure mode where new deps land - # without their lockfile entries and surface as confusing compile - # errors downstream. - mix deps.get --check-locked + # Prefer the committed lock (fast-fails if mix.exs was edited without + # a committed mix.lock). Fall back to a plain resolve so pure upstream + # drift -- a transitive dep publishing a newer patch after the lock was + # committed -- does not red the whole pipeline. The pre-commit hook and + # `mix raxol.check` still guard genuinely uncommitted lock changes. + mix deps.get --check-locked || mix deps.get mix deps.compile - name: Run tests (Unix) diff --git a/mix.lock b/mix.lock index 08d8a6d5f..7055758cf 100644 --- a/mix.lock +++ b/mix.lock @@ -66,7 +66,7 @@ "phoenix_html": {:hex, :phoenix_html, "4.3.0", "d3577a5df4b6954cd7890c84d955c470b5310bb49647f0a114a6eeecc850f7ad", [:mix], [], "hexpm", "3eaa290a78bab0f075f791a46a981bbe769d94bc776869f4f3063a14f30497ad"}, "phoenix_live_dashboard": {:hex, :phoenix_live_dashboard, "0.8.7", "405880012cb4b706f26dd1c6349125bfc903fb9e44d1ea668adaf4e04d4884b7", [:mix], [{:ecto, "~> 3.6.2 or ~> 3.7", [hex: :ecto, repo: "hexpm", optional: true]}, {:ecto_mysql_extras, "~> 0.5", [hex: :ecto_mysql_extras, repo: "hexpm", optional: true]}, {:ecto_psql_extras, "~> 0.7", [hex: :ecto_psql_extras, repo: "hexpm", optional: true]}, {:ecto_sqlite3_extras, "~> 1.1.7 or ~> 1.2.0", [hex: :ecto_sqlite3_extras, repo: "hexpm", optional: true]}, {:mime, "~> 1.6 or ~> 2.0", [hex: :mime, repo: "hexpm", optional: false]}, {:phoenix_live_view, "~> 0.19 or ~> 1.0", [hex: :phoenix_live_view, repo: "hexpm", optional: false]}, {:telemetry_metrics, "~> 0.6 or ~> 1.0", [hex: :telemetry_metrics, repo: "hexpm", optional: false]}], "hexpm", "3a8625cab39ec261d48a13b7468dc619c0ede099601b084e343968309bd4d7d7"}, "phoenix_live_reload": {:hex, :phoenix_live_reload, "1.6.2", "b18b0773a1ba77f28c52decbb0f10fd1ac4d3ae5b8632399bbf6986e3b665f62", [:mix], [{:file_system, "~> 0.2.10 or ~> 1.0", [hex: :file_system, repo: "hexpm", optional: false]}, {:phoenix, "~> 1.4", [hex: :phoenix, repo: "hexpm", optional: false]}], "hexpm", "d1f89c18114c50d394721365ffb428cce24f1c13de0467ffa773e2ff4a30d5b9"}, - "phoenix_live_view": {:hex, :phoenix_live_view, "1.2.6", "b2b50c51993b068fe2a29eda45ca4fe67bf74f146343468485686a253c65a8da", [:mix], [{:igniter, ">= 0.6.16 and < 1.0.0-0", [hex: :igniter, repo: "hexpm", optional: true]}, {:jason, "~> 1.0", [hex: :jason, repo: "hexpm", optional: true]}, {:lazy_html, "~> 0.1.0", [hex: :lazy_html, repo: "hexpm", optional: true]}, {:phoenix, "~> 1.6.15 or ~> 1.7.0 or ~> 1.8.0", [hex: :phoenix, repo: "hexpm", optional: false]}, {:phoenix_html, "~> 3.3 or ~> 4.0", [hex: :phoenix_html, repo: "hexpm", optional: false]}, {:phoenix_template, "~> 1.0", [hex: :phoenix_template, repo: "hexpm", optional: false]}, {:phoenix_view, "~> 2.0", [hex: :phoenix_view, repo: "hexpm", optional: true]}, {:plug, "~> 1.15", [hex: :plug, repo: "hexpm", optional: false]}, {:telemetry, "~> 0.4.2 or ~> 1.0", [hex: :telemetry, repo: "hexpm", optional: false]}], "hexpm", "5dc549a78dab94e80a340c760090e9e7bab16ac47841e34f4fff5819e02cbf35"}, + "phoenix_live_view": {:hex, :phoenix_live_view, "1.2.7", "d0f20871681216598e78baccd4f66d8686fdb8007821989bab508d293a3ed9ad", [:mix], [{:igniter, ">= 0.6.16 and < 1.0.0-0", [hex: :igniter, repo: "hexpm", optional: true]}, {:jason, "~> 1.0", [hex: :jason, repo: "hexpm", optional: true]}, {:lazy_html, "~> 0.1.0", [hex: :lazy_html, repo: "hexpm", optional: true]}, {:phoenix, "~> 1.6.15 or ~> 1.7.0 or ~> 1.8.0", [hex: :phoenix, repo: "hexpm", optional: false]}, {:phoenix_html, "~> 3.3 or ~> 4.0", [hex: :phoenix_html, repo: "hexpm", optional: false]}, {:phoenix_template, "~> 1.0", [hex: :phoenix_template, repo: "hexpm", optional: false]}, {:phoenix_view, "~> 2.0", [hex: :phoenix_view, repo: "hexpm", optional: true]}, {:plug, "~> 1.15", [hex: :plug, repo: "hexpm", optional: false]}, {:telemetry, "~> 0.4.2 or ~> 1.0", [hex: :telemetry, repo: "hexpm", optional: false]}], "hexpm", "61e97938a4fcca6d6f2c836925623abf2f52a572cc8c6085e4074f3f6337e0eb"}, "phoenix_pubsub": {:hex, :phoenix_pubsub, "2.2.0", "ff3a5616e1bed6804de7773b92cbccfc0b0f473faf1f63d7daf1206c7aeaaa6f", [:mix], [], "hexpm", "adc313a5bf7136039f63cfd9668fde73bba0765e0614cba80c06ac9460ff3e96"}, "phoenix_template": {:hex, :phoenix_template, "1.0.4", "e2092c132f3b5e5b2d49c96695342eb36d0ed514c5b252a77048d5969330d639", [:mix], [{:phoenix_html, "~> 2.14.2 or ~> 3.0 or ~> 4.0", [hex: :phoenix_html, repo: "hexpm", optional: true]}], "hexpm", "2c0c81f0e5c6753faf5cca2f229c9709919aba34fab866d3bc05060c9c444206"}, "plug": {:hex, :plug, "1.20.3", "56c480c633ec2ce10140e236e15233bf576e1d323887d7c96711bd02ab5160db", [:mix], [{:mime, "~> 1.0 or ~> 2.0", [hex: :mime, repo: "hexpm", optional: false]}, {:plug_crypto, "~> 1.1.1 or ~> 1.2 or ~> 2.0", [hex: :plug_crypto, repo: "hexpm", optional: false]}, {:telemetry, "~> 0.4.3 or ~> 1.0", [hex: :telemetry, repo: "hexpm", optional: false]}], "hexpm", "be266aee1b8536ef6409d58cf39a3121319f0ec47cfa1b24024485aa0e76ad76"},