commit 3683d6b7a85c97d067c18fc283bf1c490087a8a9
parent 59542504b41dfddb6f26524fb4ce811e0f22a785
Author: dundargoc <33953936+dundargoc@users.noreply.github.com>
Date: Sat, 4 Mar 2023 17:55:42 +0100
ci: move configuring cmake to the build part of the CI
If the configuration fails then lints shouldn't be run, as most lint
steps depends on a successful configuration.
Diffstat:
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
@@ -75,6 +75,10 @@ jobs:
cmake -S cmake.deps -B $DEPS_BUILD_DIR -G Ninja
cmake --build $DEPS_BUILD_DIR
+ - if: success() || failure() && steps.abort_job.outputs.status == 'success'
+ name: configure
+ run: cmake -B build -G Ninja -D CI_BUILD=OFF
+
- if: "!cancelled()"
name: Determine if run should be aborted
id: abort_job
@@ -89,10 +93,6 @@ jobs:
args: --check runtime/
- if: success() || failure() && steps.abort_job.outputs.status == 'success'
- name: configure
- run: cmake -B build -G Ninja -D CI_BUILD=OFF
-
- - if: success() || failure() && steps.abort_job.outputs.status == 'success'
name: luacheck
run: cmake --build build --target lintlua-luacheck