Skip to content

Conversation

emilk
Copy link
Owner

@emilk emilk commented Oct 2, 2025

No description provided.

@emilk emilk requested a review from Copilot October 2, 2025 17:17
@emilk emilk added CI Continues Integration test exclude from changelog labels Oct 2, 2025
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR replaces cargo check commands with cargo clippy in the CI workflow to provide more comprehensive code analysis and linting during continuous integration. The change upgrades the static analysis from basic compilation checks to include clippy's lint warnings.

  • Replaced all cargo check commands with cargo clippy equivalents across different target configurations
  • Removed redundant step names for consistency, using inline run commands
  • Maintained all existing flags and package specifications for each clippy command

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Copy link

github-actions bot commented Oct 2, 2025

Preview available at https://egui-pr-preview.github.io/pr/7581-emilkmore-clippy-on-ci
Note that it might take a couple seconds for the update to show up after the preview_build workflow has completed.

@emilk emilk force-pushed the emilk/more-clippy-on-ci branch from 02f9958 to 1792f46 Compare October 2, 2025 17:31
@emilk emilk requested a review from Wumpf as a code owner October 2, 2025 17:31
@emilk emilk force-pushed the emilk/more-clippy-on-ci branch from 2edfe4d to 9e7a50d Compare October 2, 2025 17:59
@emilk emilk force-pushed the emilk/more-clippy-on-ci branch from 9e7a50d to ac9e7af Compare October 2, 2025 18:10
@emilk emilk merged commit 096ed1c into main Oct 2, 2025
47 checks passed
@emilk emilk deleted the emilk/more-clippy-on-ci branch October 2, 2025 18:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI Continues Integration test exclude from changelog

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant