Skip to content

test_auto_formate #6247

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
Piyush1045 wants to merge 2 commits into from
Closed

test_auto_formate #6247

Piyush1045 wants to merge 2 commits into from

Conversation

@Piyush1045
Copy link

@Piyush1045 Piyush1045 commented Nov 10, 2025 •
edited by coderabbitai bot
Loading

Summary by CodeRabbit

  • Style
    • Updated internal build configuration formatting for improved code readability.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Nov 10, 2025 •
edited
Loading

Walkthrough

The build.rs file receives a formatting update. A multi-line else branch is collapsed to a single line, and the res.compile() call is reformatted with chained map_err. Functionality remains unchanged.

Changes

Cohort / File(s) Change Summary
Build Script Formatting
build.rs		 Simplified else branch to single-line format; reformatted res.compile() call to chain map_err on the same line for improved readability

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

  • Purely cosmetic formatting adjustments with no semantic changes
  • No logic, control flow, or error handling modifications

Poem

🐰 A tidy build script hops along,
With formatted lines, neat and strong,
No logic bent, just cleaner views—
A rabbit's touch on builder's muse! ✨

Pre-merge checks and finishing touches

❌ Failed checks (2 warnings)
Check name Status Explanation Resolution
Title check ⚠️ Warning The title 'test_auto_formate' does not clearly describe the actual changes, which involve formatting adjustments in build.rs (collapsing multi-line else branch and reformatting res.compile() call). Use a descriptive title that reflects the actual change, such as 'Format build.rs: collapse else branch and chain map_err' or similar.
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. You can run @coderabbitai generate docstrings to improve docstring coverage.
✅ Passed checks (1 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
📜 Recent review details

Configuration used: Path: .coderabbit.yml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 2d46172 and 2f5515f.

📒 Files selected for processing (1)
  • build.rs (1 hunks)
🧰 Additional context used
📓 Path-based instructions (1)
**/*.rs

📄 CodeRabbit inference engine (.github/copilot-instructions.md)

**/*.rs: Format Rust code with the default rustfmt style (run cargo fmt)
Run clippy and fix any warnings or lints introduced by your changes
Follow Rust best practices for error handling and memory management

Files:

  • build.rs
🧠 Learnings (4)
📓 Common learnings 
Learnt from: CR
Repo: RustPython/RustPython PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-09-11T05:06:59.971Z
Learning: Applies to Lib/test/**/*.py : When modifying CPython tests, add a '# TODO: RUSTPYTHON' comment documenting the reason

Learnt from: CR
Repo: RustPython/RustPython PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-09-11T05:06:59.971Z
Learning: Applies to **/*.rs : Format Rust code with the default rustfmt style (run `cargo fmt`)

Learnt from: CR
Repo: RustPython/RustPython PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-09-11T05:06:59.971Z
Learning: Applies to **/*.rs : Follow Rust best practices for error handling and memory management

Learnt from: CR
Repo: RustPython/RustPython PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-09-11T05:06:59.971Z
Learning: Applies to **/*.rs : Run clippy and fix any warnings or lints introduced by your changes

Learnt from: arihant2math
Repo: RustPython/RustPython PR: 5790
File: build.rs:2-2
Timestamp: 2025-06-28T16:31:03.991Z
Learning: In Cargo build scripts (build.rs), the environment variable CARGO_CFG_TARGET_OS is guaranteed to exist and is automatically set by Cargo during the build process, making unwrap() safe to use when accessing this variable.
📚 Learning: 2025-09-11T05:06:59.971Z 
Learnt from: CR
Repo: RustPython/RustPython PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-09-11T05:06:59.971Z
Learning: Applies to **/*.rs : Run clippy and fix any warnings or lints introduced by your changes

Applied to files:

  • build.rs
📚 Learning: 2025-09-11T05:06:59.971Z 
Learnt from: CR
Repo: RustPython/RustPython PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-09-11T05:06:59.971Z
Learning: Applies to **/*.rs : Format Rust code with the default rustfmt style (run `cargo fmt`)

Applied to files:

  • build.rs
📚 Learning: 2025-09-11T05:06:59.971Z 
Learnt from: CR
Repo: RustPython/RustPython PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-09-11T05:06:59.971Z
Learning: Applies to **/*.rs : Follow Rust best practices for error handling and memory management

Applied to files:

  • build.rs
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
  • GitHub Check: auto_format
🔇 Additional comments (1)
build.rs (1)

7-13: Run cargo fmt locally to correct the formatting in vm/build.rs.

The formatting at lines 7-13 in vm/build.rs does not match standard Rust formatting conventions:

  • Line 7: The else block should not have the println! collapsed to the same line as the opening brace. Standard cargo fmt places each statement on a separate indented line:

    } else {
    println!("cargo:warning=logo.ico not found, skipping icon embedding");
    return;
}

  • Lines 10-13: The indentation pattern for the res.compile().map_err(...) chain and closure body appears non-standard.

Since the PR title references auto-formatting verification, these changes must conform to cargo fmt output. Run cargo fmt locally on vm/build.rs and commit the corrected formatting before merging.

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions
Copy link
Contributor

github-actions bot commented Nov 10, 2025

Code has been automatically formatted

The code in this PR has been formatted using cargo fmt.
The changes have been committed with [skip ci] to avoid triggering another CI run.

Triggered by commit: 2f5515f3c22447b6741000e1942cc87edf656427
Last formatted: 2025-11-10T19:06:04Z

You may need to pull the latest changes before pushing again:

git pull origin main

@Piyush1045
Copy link
Author

Piyush1045 commented Nov 10, 2025

@youknowone here it passes

@fanninpm
Copy link
Contributor

fanninpm commented Nov 10, 2025

@Piyush1045 this is because you're pushing from your main branch, which I don't recommend.

@YashSuthar983 YashSuthar983 mentioned this pull request Nov 10, 2025
Merged
@Piyush1045 Piyush1045 closed this Nov 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@Piyush1045 @fanninpm