Skip to content

Commit ca98674

Browse files
committed
ci: run clippy on all crates
1 parent 34b4e95 commit ca98674

File tree

3 files changed

+14
-36
lines changed

3 files changed

+14
-36
lines changed

.github/features.ua

Lines changed: 0 additions & 20 deletions
This file was deleted.

.github/workflows/validate.yml

Lines changed: 13 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -42,21 +42,29 @@ jobs:
4242
- uses: actions/checkout@v4
4343
- name: Check formatting
4444
run: cargo fmt --all --check
45-
install_deps:
46-
needs: check_formatting
45+
clippy:
4746
runs-on: ubuntu-latest
4847
steps:
4948
- uses: actions/checkout@v4
50-
- name: Install depsAdd commentMore actions
49+
- uses: actions/cache@v4
50+
with:
51+
path: target
52+
key: build-${{ runner.os }}-${{ hashFiles('Cargo.lock') }}
53+
restore-keys: |
54+
build-${{ runner.os }}-
55+
- name: Install deps
5156
run: |
5257
sudo apt-get update -y
5358
sudo apt-get --fix-missing install -y
54-
sudo apt-get install libasound2-dev libudev-dev pkg-config libx11-dev libjpeg-dev
59+
sudo apt-get install libasound2-dev pkg-config libx11-dev libjpeg-dev
5560
- name: Install WASM target
5661
run: rustup target add wasm32-unknown-unknown
62+
- name: Run clippy
63+
run: |
64+
cargo clippy --no-deps --all-targets --workspace --all-features -- --deny=warnings
65+
cargo clippy --no-deps --all-targets --workspace --all-features --target wasm32-unknown-unknown -- --deny=warnings
5766
# Test the interpreter
5867
test_bin:
59-
needs: install_deps
6068
runs-on: ubuntu-latest
6169
steps:
6270
- uses: actions/checkout@v4
@@ -66,13 +74,10 @@ jobs:
6674
key: build-${{ runner.os }}-${{ hashFiles('Cargo.lock') }}
6775
restore-keys: |
6876
build-${{ runner.os }}-
69-
- name: Check interpreter clippy lints
70-
run: cargo clippy --no-deps
7177
- name: Run interpreter tests
7278
run: cargo test --lib
7379
# Test the site
7480
test_site:
75-
needs: install_deps
7681
runs-on: ubuntu-latest
7782
steps:
7883
- uses: actions/checkout@v4
@@ -86,7 +91,6 @@ jobs:
8691
run: cargo test -p site
8792
# Test ffi
8893
test_ffi:
89-
needs: install_deps
9094
runs-on: ubuntu-latest
9195
steps:
9296
- uses: actions/checkout@v4
@@ -98,10 +102,3 @@ jobs:
98102
build-${{ runner.os }}-
99103
- name: Run site tests
100104
run: cargo test -p tests_ffi
101-
# check_features:
102-
# needs: test_bin
103-
# runs-on: ubuntu-latest
104-
# steps:
105-
# - uses: actions/checkout@v4
106-
# - name: Check feature combinations
107-
# run: cargo run ./.github/features.ua

Cargo.toml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -165,6 +165,7 @@ system = ["libffi?/system"]
165165

166166
[[bin]]
167167
name = "uiua"
168+
required-features = ["binary"]
168169

169170
[workspace]
170171
members = ["site", "tests_ffi", "pad/editor", "parser"]

0 commit comments

Comments
 (0)