Skip to content

Commit

Permalink
ci: Fix rust-clippy job
Browse files Browse the repository at this point in the history
Add temporary install step for missing libcurl/libstb and ensure all
features are checked separately

Signed-off-by: Kostis Papazafeiropoulos <[email protected]>
  • Loading branch information
papazof committed Jan 31, 2025
1 parent ce6a783 commit 1fc9212
Showing 1 changed file with 18 additions and 1 deletion.
19 changes: 18 additions & 1 deletion .github/workflows/validate-code.yml
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,9 @@ on:
runner-arch-map:
type: string
default: '[{"amd64":"x86_64", "arm64":"aarch64", "arm":"armv7l"}]'
clippy-features:
type: string
default: 'async,async-stream'
secrets:
GIT_CLONE_PAT:
required: false
Expand Down Expand Up @@ -59,6 +62,13 @@ jobs:
remote-actions-repo: ${{ inputs.actions-repo }}
token: ${{ secrets.GIT_CLONE_PAT || github.token }}

# FIXME: Temp
- name: Install libcurl & stb
run: |
sudo apt-get update
sudo apt-get install -y libcurl4-openssl-dev libstb-dev
shell: bash

- name: Download artifacts from s3
id: download-artifacts
if: ${{ inputs.actions-repo != github.repository }}
Expand All @@ -73,7 +83,14 @@ jobs:
install: 'true'

- name: Run rust-clippy
run: cargo clippy --workspace --all-targets --all-features
run: |
cargo clippy --workspace --all-targets
cargo clippy --workspace --all-targets --all-features
IFS=','
features=${{ inputs.clippy-features }}
for f in $features; do
cargo clippy --workspace --all-targets --features "$f"
done
shell: bash

- name: Clean-up
Expand Down

0 comments on commit 1fc9212

Please sign in to comment.