Skip to content

Commit da3f408

Browse files
authored
bridge: Remove build step from CI workflow (#1347)
There is no need to run `cargo build` separately from `cargo test` and `cargo clippy`. The former command already builds the library with and without `cfg(test)` (for unit tests and integration tests, respectively). Also rename some CI steps to make it more obvious what they're doing.
2 parents a8f2b8f + b91a3bc commit da3f408

File tree

1 file changed

+7
-11
lines changed

1 file changed

+7
-11
lines changed

.github/workflows/bridge-ci.yml

Lines changed: 7 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -63,7 +63,11 @@ jobs:
6363

6464
- uses: taiki-e/install-action@nextest
6565

66-
- name: Install dependencies
66+
- name: Clippy
67+
run: cargo clippy --locked --all-targets --all-features -- -D warnings
68+
working-directory: bridge
69+
70+
- name: Install system dependencies
6771
# Packages should align with whatever is in the bridge/Dockerfile
6872
run: |
6973
sudo apt-get install -y \
@@ -74,22 +78,14 @@ jobs:
7478
pkg-config=0.29.* \
7579
zlib1g-dev=1:*
7680
77-
- name: Build
78-
run: cargo build --locked
79-
working-directory: bridge
80-
81-
- name: Start dependencies
81+
- name: Start required services
8282
run: docker compose -f "bridge/testing-docker-compose.yml" up -d
8383

84-
- name: Clippy
85-
run: cargo clippy --all-targets --all-features -- -D warnings
86-
working-directory: bridge
87-
8884
- name: Run tests
8985
working-directory: bridge
9086
run: ./run-tests.sh
9187

92-
- name: Stop dependencies
88+
- name: Stop required services
9389
run: docker compose -f "bridge/testing-docker-compose.yml" down
9490

9591
# deny-check:

0 commit comments

Comments
 (0)