Move from CircleCI to Github Actions

.github/workflows/ci.yml

@@ -0,0 +1,33 @@
+on: push
+
+env:
+  ELIXIR_ERL_OPTIONS: -kernel prevent_overlapping_partitions false
+
+jobs:
+  test:
+    runs-on: ubuntu-latest
+    strategy:
+      matrix:
+        elixir-version: [1.15, 1.14, 1.13]
+        otp-version: [26, 25, 24]
+        exclude:
+          - elixir-version: 1.13
+            otp-version: 26
+
+    name: Tests
+    steps:
+      - uses: actions/checkout@v3
+      - uses: erlef/setup-beam@v1
+        with:
+          elixir-version: ${{ matrix.elixir-version }}
+          otp-version: ${{ matrix.otp-version }}
+
+      - run: mix deps.get
+
+      - run: mix compile --warnings-as-errors
+        if: matrix.elixir-version == 1.15 && matrix.otp-version == 26
+
+      - run: mix format --check-formatted
+        if: matrix.elixir-version == 1.15 && matrix.otp-version == 26
+
+      - run: mix test

lib/horde/processes_supervisor.ex

@@ -822,7 +822,8 @@ defmodule Horde.ProcessesSupervisor do
   end
 
   def handle_info(msg, state) do
-    :error_logger.error_msg('Horde.ProcessesSupervisor received unexpected message: ~p~n', [msg])
+    :error_logger.error_msg(~c"Horde.ProcessesSupervisor received unexpected message: ~p~n", [msg])
+
     {:noreply, state}
   end
 

lib/horde/supervisor_telemetry_poller.ex

@@ -34,7 +34,7 @@ defmodule Horde.DynamicSupervisorTelemetryPoller do
     })
   catch
     :exit, reason ->
-      Logger.warn("""
+      Logger.warning("""
       Exit while fetching metrics from #{inspect(supervisor_impl_name)}.
       Skip poll action. Reason: #{inspect(reason)}.
       """)

mix.exs

@@ -38,7 +38,7 @@ defmodule Horde.MixProject do
       {:stream_data, "~> 0.4", only: :test},
       {:local_cluster, "~> 1.1", only: :test},
       {:schism, "~> 1.0.1", only: :test},
-      {:dialyxir, "~> 1.0.0-rc.6", only: [:dev, :test], runtime: false},
+      {:dialyxir, "~> 1.0.0-rc.6", only: [:dev], runtime: false},
       {:test_app, path: "test_app", only: [:test]}
     ]
   end

test/local_cluster_test.exs

@@ -82,7 +82,7 @@
       pid when is_pid(pid) ->
         pid
 
       x ->
         Process.sleep(200)
         do_await_replication(target, id)
     end
@@ -101,7 +101,7 @@
             nodes = rpc(target, Node, :list, [])
 
             Logger.info(
-              "found #{inspect(pid)} on node: #{inspect(node(pid))}, target #{inspect(node)}, nodes: #{inspect(nodes)}"
+              "found #{inspect(pid)} on node: #{inspect(node(pid))}, target #{inspect(node())}, nodes: #{inspect(nodes)}"
             )
 
             Process.sleep(200)

test/network_partition_test.exs

@@ -2,7 +2,7 @@ defmodule NetworkPartitionTest do
   use ExUnit.Case
 
   setup do
-    nodes = LocalCluster.start_nodes("cluster-#{:rand.uniform(9_999_999)}", 2)
+    nodes = LocalCluster.start_nodes("cluster#{:erlang.unique_integer()}", 2)
 
     for n <- nodes do
       :rpc.call(n, Application, :ensure_all_started, [:test_app])
@@ -12,13 +12,13 @@ defmodule NetworkPartitionTest do
   end
 
   test "recovers as expected in case of network partition", %{nodes: [n1, n2] = nodes} do
-    assert {:ok, pid1} =
+    assert {:ok, _pid1} =
              Horde.DynamicSupervisor.start_child(
                {TestSup, n1},
                {IgnoreWorker, {:via, Horde.Registry, {TestReg, IgnoreWorker}}}
              )
 
-    assert {:ok, pid2} =
+    assert {:ok, _pid2} =
              Horde.DynamicSupervisor.start_child(
                {TestSup, n2},
                {IgnoreWorker, {:via, Horde.Registry, {TestReg, IgnoreWorker}}}
@@ -48,13 +48,13 @@ defmodule NetworkPartitionTest do
   end
 
   test "recovers as expected in case of node stopping", %{nodes: [n1, n2] = nodes} do
-    assert {:ok, pid1} =
+    assert {:ok, _pid1} =
              Horde.DynamicSupervisor.start_child(
                {TestSup, n1},
                {IgnoreWorker, {:via, Horde.Registry, {TestReg, IgnoreWorker}}}
              )
 
-    assert {:ok, pid2} =
+    assert {:ok, _pid2} =
              Horde.DynamicSupervisor.start_child(
                {TestSup, n2},
                {IgnoreWorker, {:via, Horde.Registry, {TestReg, IgnoreWorker}}}