Skip to content

Commit

Permalink
ci: remove main_tests from matrix
Browse files Browse the repository at this point in the history
It is set to `1` all the time anyway.
  • Loading branch information
pvdrz committed Dec 4, 2024
1 parent 798d618 commit 2da868d
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 6 deletions.
2 changes: 0 additions & 2 deletions .github/workflows/bindgen.yml
Original file line number Diff line number Diff line change
Expand Up @@ -126,7 +126,6 @@ jobs:
matrix:
os: [ubuntu-latest]
llvm_version: ["9.0", "16.0"]
main_tests: [1]
release_build: [0, 1]
no_default_features: [0, 1]
# FIXME: There are no pre-built static libclang libraries, so the
Expand Down Expand Up @@ -168,7 +167,6 @@ jobs:
- name: Run all the tests
env:
GITHUB_ACTIONS_OS: ${{matrix.os}}
BINDGEN_MAIN_TESTS: ${{matrix.main_tests}}
BINDGEN_RELEASE_BUILD: ${{matrix.release_build}}
BINDGEN_FEATURE_RUNTIME: ${{matrix.feature_runtime}}
BINDGEN_FEATURE_EXTRA_ASSERTS: ${{matrix.feature_extra_asserts}}
Expand Down
6 changes: 2 additions & 4 deletions ci/test.sh
Original file line number Diff line number Diff line change
Expand Up @@ -53,10 +53,8 @@ CARGO_ARGS=`get_cargo_args`
# Ensure we build without warnings
RUSTFLAGS="-Dwarnings" cargo check $CARGO_ARGS

if [ "$BINDGEN_MAIN_TESTS" == "1" ]; then
# Run the tests
(cd bindgen-tests && cargo test $CARGO_ARGS)
fi
# Run the tests
(cd bindgen-tests && cargo test $CARGO_ARGS)

assert_no_diff

Expand Down

0 comments on commit 2da868d

Please sign in to comment.