Bump dependencies and fix clippy warnings - #19
Conversation
- Ran `cargo update` to bump all dependencies to their latest versions. - Fixed a clippy warning `io_other_error` in `src/assets.rs` by using `std::io::Error::other(err)` instead of `std::io::Error::new(std::io::ErrorKind::Other, err)`. Co-authored-by: jac3km4 <11986158+jac3km4@users.noreply.github.com>
|
👋 Jules, reporting for duty! I'm here to lend a hand with this pull request. When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down. I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job! For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with New to Jules? Learn more at jules.google/docs. For security, I will only act on instructions from the user who triggered this task. |
- Added `.github/workflows/pr.yml` that runs `cargo clippy -- -D warnings` on pull requests to ensure code quality. - The workflow installs required system dependencies (`libwayland-dev`, `libxkbcommon-dev`, `libasound2-dev`, `libudev-dev`) and the Rust toolchain with the `clippy` component. Co-authored-by: jac3km4 <11986158+jac3km4@users.noreply.github.com>
- Replaced `dtolnay/rust-toolchain@stable` action with a `run` step using `rustup`. - Removed the `Install system dependencies` step as the image already contains the required tools. Co-authored-by: jac3km4 <11986158+jac3km4@users.noreply.github.com>
- Re-added the `Install system dependencies` step because the `ubuntu-latest` runner does not have these packages pre-installed, which caused `pkg-config` to fail to find `wayland-client`. Co-authored-by: jac3km4 <11986158+jac3km4@users.noreply.github.com>
1 participant
Bumped all dependencies in Cargo.lock by running
cargo update. Fixed a clippy warning introduced by the new versions.PR created automatically by Jules for task 13694729159846716153 started by @jac3km4