Skip to content

Commit

Permalink
Do not build & run unit tests
Browse files Browse the repository at this point in the history
Fixes the problem of unit tests spamming benchmark output, making it unparseable. Also makes things a bit leaner.

CI on the main Luanti repo already ensures that these tests pass on sufficiently similar configurations.
  • Loading branch information
appgurueu committed Jan 29, 2025
1 parent 451fffd commit 9bf4ff2
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion .github/workflows/track_minetest.yml
Original file line number Diff line number Diff line change
Expand Up @@ -58,7 +58,8 @@ jobs:
CMAKE_BUILD_TYPE: "Release"
CMAKE_BUILD_SERVER: FALSE
CMAKE_ENABLE_GETTEXT: FALSE
CMAKE_FLAGS: "-DBUILD_BENCHMARKS=TRUE"
# Do not build tests: Tests may add spam to the benchmark output, letting parsing fail.
CMAKE_FLAGS: "-DBUILD_BENCHMARKS=TRUE -DBUILD_UNITTESTS=FALSE"
- name: Run Benchmarks
if: ${{ env.DID_PUSH == 'true' }}
run: |
Expand Down

0 comments on commit 9bf4ff2

Please sign in to comment.