Skip to content

Commit 3055a09

Browse files
committed
fix(ci): move test --cover out of mix ci alias
mix ci previously included 'test --cover' as its last step, which caused an Elixir 1.20 warning about running in the wrong environment. Tests are already standalone steps in the GitHub Actions CI workflow (both Linux and macOS runners). Changes: * Removed 'test --cover' from the ci alias * Removed preferred_envs from def project (no longer needed since ci no longer chains into test) * Removed empty def cli block mix ci now runs 7 steps: compile, format, credo, deps check, audit, xref, dialyzer, ex_dna. Tests are run separately with MIX_ENV=test.
1 parent 00c8d38 commit 3055a09

1 file changed

Lines changed: 0 additions & 5 deletions

File tree

mix.exs

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -155,7 +155,6 @@ defmodule AdoCli.MixProject do
155155
"deps.audit",
156156
"xref graph --label compile-connected --fail-above 0",
157157
"ci.dialyzer",
158-
"test --cover",
159158
"ex_dna"
160159
],
161160
quality: [
@@ -171,10 +170,6 @@ defmodule AdoCli.MixProject do
171170
]
172171
end
173172

174-
def cli do
175-
[preferred_envs: [test: :test, "test --cover": :test, "test.cover": :test, "coveralls.json": :test]]
176-
end
177-
178173
defp escript_config do
179174
[
180175
main_module: AdoCli.CLI,

0 commit comments

Comments
 (0)