Skip to content

Conversation

magravel
Copy link
Contributor

@magravel magravel commented Aug 1, 2025

Adds a Rust workflow that makes sure the code is well formatted, doesn't have Clippy errors, and that tests that are passing and validate the documentation.

This workflow is synced with these repositories:

  • microsoft/mu_basecore
  • microsoft/mu_plus
  • microsoft/mu_rust_helpers
  • microsoft/mu_rust_hid
  • microsoft/mu_rust_pi
  • microsoft/mu_tiano_platforms

This has been tested on a dummy repository.

Example: When fmt fails, we want other checks to still run in that case.
image

@makubacki
Copy link
Member

The workflow does not need to run in this repo. You can place it in the https://github.com/microsoft/mu_devops/tree/main/.sync/workflows/leaf so it will be synced to repos with Rust code.

@magravel magravel marked this pull request as ready for review August 1, 2025 18:33
@Javagedes Javagedes linked an issue Aug 4, 2025 that may be closed by this pull request
@magravel magravel requested review from makubacki and Javagedes August 4, 2025 21:08
@makubacki
Copy link
Member

@magravel, are you planning to let the file sync PR be created in all of these repos and then amend the fixes needed to the bot's file sync PR?

@magravel
Copy link
Contributor Author

magravel commented Aug 4, 2025

@magravel, are you planning to let the file sync PR be created in all of these repos and then amend the fixes needed to the bot's file sync PR?

Yes

@makubacki
Copy link
Member

@magravel, since we're expecting sync PRs to fail and need to be fixed. Let's wait until Monday to merge this so it doesn't affect anything sync changes that may need to go in in the meantime.

@makubacki makubacki changed the title rust-workflow Add RustCargoSteps workflow Aug 18, 2025
@makubacki makubacki merged commit 04732fd into microsoft:main Aug 18, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add Cargo Clippy to rust steps
4 participants