Skip to content

Conversation

@George-Miao
Copy link
Member

No description provided.

@George-Miao George-Miao requested a review from Berrysoft October 27, 2025 21:32
@George-Miao George-Miao self-assigned this Oct 27, 2025
@George-Miao George-Miao requested a review from Copilot October 27, 2025 21:32
Copy link
Contributor

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 adds a new GitHub Actions workflow to enforce code formatting standards in the CI pipeline. The changes include the addition of a new workflow file and automatic formatting fixes to existing code, bringing them into compliance with the project's Rust formatting guidelines.

Key Changes

  • Added a new GitHub Actions workflow (ci_format.yml) to check code formatting using cargo fmt
  • Reformatted import statements and comments across multiple files to comply with formatting standards

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

File Description
.github/workflows/ci_format.yml New CI workflow that runs cargo fmt --check on push/PR to master branch
compio-runtime/tests/drop.rs Reorganized imports to place external crate imports after standard library imports
compio-runtime/src/runtime/scheduler/mod.rs Reordered imports and reformatted multi-line comments to meet line length requirements
compio-fs/src/file.rs Reformatted doc comment to comply with line length standards

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@George-Miao George-Miao force-pushed the ci/fmt branch 3 times, most recently from 259a4b8 to f4676cc Compare October 27, 2025 21:41
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@Berrysoft Berrysoft merged commit e7d75e6 into master Oct 28, 2025
51 checks passed
@Berrysoft Berrysoft deleted the ci/fmt branch October 28, 2025 04:42
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.

3 participants